kaxil commented on a change in pull request #12745:
URL: https://github.com/apache/airflow/pull/12745#discussion_r533692462
##########
File path: airflow/models/dagrun.py
##########
@@ -724,7 +724,7 @@ def schedule_tis(self, schedulable_tis: Iterable[TI],
session: Session = None) -
ti
for ti in schedulable_tis
if (
- ti.task.task_type == "DummyOperator"
+ (ti.task.task_type == "DummyOperator" or getattr(ti.task,
"_is_dummy", False))
Review comment:
This is just for backwards-compatibility so the current Serialized DAGs
which don't have `_is_dummy` field still continue to work
##########
File path: airflow/serialization/serialized_objects.py
##########
@@ -432,6 +437,9 @@ def deserialize_operator(cls, encoded_op: Dict[str, Any])
-> BaseOperator:
if not hasattr(op, field):
setattr(op, field, None)
+ # Used to determine if an Operator is inherited from DummyOperator
+ setattr(op, "_is_dummy", bool(encoded_op.get("_is_dummy")))
Review comment:
fixed in
https://github.com/apache/airflow/pull/12745/commits/67b5daf93d7a791670966cea806429155a303041
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
