ephraimbuddy commented on code in PR #21877:
URL: https://github.com/apache/airflow/pull/21877#discussion_r843214681
##########
airflow/jobs/scheduler_job.py:
##########
@@ -783,6 +783,22 @@ def _execute(self) -> None:
self.log.exception("Exception when executing
DagFileProcessorAgent.end")
self.log.info("Exited execute loop")
+ def _update_dag_run_state_for_paused_dags(self):
+ try:
+ paused_dag_ids = DagModel.get_all_paused_dag_ids()
+ for dag_id in paused_dag_ids:
Review Comment:
It's possible that at the first check of the `dag_run`, the `dag_run` state
could not be updated because the task instances are still running. If we cache
the dag, and the task instances of the `dag_run` `complete` later on, we won't
be able to update the `dagrun` state, right?
--
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]
