potiuk commented on code in PR #21877:
URL: https://github.com/apache/airflow/pull/21877#discussion_r871210470
##########
airflow/jobs/scheduler_job.py:
##########
@@ -779,6 +779,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):
Review Comment:
I quite agree with no need to update pasused tasks state immediately - this
makes this part of the changfe far more appealing. Since this change is merged
- @ashb - if you stil have some concerns, we can continue discussing here and
maybe make a follow-up change in case we think of a scenario where it might be
problematic.
--
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]
