TSKVenkat commented on PR #72250: URL: https://github.com/apache/airflow/pull/72250#issuecomment-5505095793
Gentle ping, per the 72-hour guidance in the contributors' guide. This is waiting on a maintainer to approve the CI workflow run — only 2 checks have executed so far, so the full suite has not had a chance to report. Happy to rebase or adjust anything if that would help. Quick recap for whoever picks it up: changing a Dag from an hourly to a daily cron stops it being scheduled again, because the timetable proposes a `logical_date` the previous run already occupies and the scheduler loops on `run already exists; skipping dagrun creation`. #66132 added a guard for this but only for the zero-length intervals `CronTriggerTimetable` leaves behind; the same collision happens after any switch to a coarser cron. The change generalises that existing guard, so the test added with #66132 still passes untouched. No rush at all — just flagging it since it had gone quiet. -- 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]
