kacpermuda commented on code in PR #56039:
URL: https://github.com/apache/airflow/pull/56039#discussion_r2375347283
##########
airflow-core/src/airflow/models/dagrun.py:
##########
@@ -1977,6 +1977,36 @@ def get_latest_runs(cls, session: Session = NEW_SESSION)
-> list[DagRun]:
)
).all()
+ @staticmethod
+ def is_schedulable_task(task: Operator) -> bool:
Review Comment:
Wanted to keep it close to the current code, where the decision about the
scheduling is made. I'm not up to date with decoupling of core and sdk, I know
that there were different models the DagRun can receive here, Serialized
Operator, Mapped etc. so I thought it's better to have it on DagRun. I'm okay
with moving it to another place. Where exactly would you see it fit better so
that the listeners can take advantage of it both on scheduler and on worker?
--
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]
