kacpermuda commented on code in PR #56039:
URL: https://github.com/apache/airflow/pull/56039#discussion_r2447955209


##########
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:
   > OL logic is run in the worker, so you won’t be able to access this logic 
in the OL provider no matter where you put it
   
   The dagrun listener hook runs on scheduler, so I should be able to retrieve 
that information ? We're 
[re-creating](https://github.com/apache/airflow/blob/main/providers/openlineage/src/airflow/providers/openlineage/utils/utils.py#L803)
 this check in OL anyway, so I simply wanted to extract that to a separate 
method and call it. 
   



-- 
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]

Reply via email to