kyungjunleeme commented on code in PR #53937:
URL: https://github.com/apache/airflow/pull/53937#discussion_r2250265789
##########
task-sdk/src/airflow/sdk/definitions/_internal/abstractoperator.py:
##########
@@ -94,8 +94,9 @@ class AbstractOperator(Templater, DAGNode):
interface. Unfortunately it's difficult to use this as the common base
class
for typing due to BaseOperator carrying too much historical baggage.
- The union type ``from airflow.models.operator import Operator`` is easier
- to use for typing purposes.
+ For internal typing purposes, use ``airflow.sdk.types.Operator`` — a union
+ type representing both mapped and unmapped operators. This is not part of
+ the public API and should not be imported directly by users.
Review Comment:
While working on this change, I noticed a docstring that should be revised
to better reflect the current usage and prevent confusion.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
