Skip to content

Fix Dag run scheduling after coarser schedule changes#66761

Open
anmolxlight wants to merge 1 commit into
apache:mainfrom
anmolxlight:fix-66754-schedule-change-dagrun
Open

Fix Dag run scheduling after coarser schedule changes#66761
anmolxlight wants to merge 1 commit into
apache:mainfrom
anmolxlight:fix-66754-schedule-change-dagrun

Conversation

@anmolxlight
Copy link
Copy Markdown
Contributor

Summary

  • prevent data-interval timetables from re-emitting an existing logical date after a Dag schedule changes to a coarser cadence
  • keep the existing zero-length interval guard working while covering hourly-to-daily schedule changes
  • add a regression test for the issue scenario

Fixes #66754

Tests

  • uv run ruff format airflow-core/src/airflow/timetables/interval.py airflow-core/tests/unit/timetables/test_interval_timetable.py
  • uv run ruff check --fix airflow-core/src/airflow/timetables/interval.py airflow-core/tests/unit/timetables/test_interval_timetable.py
  • uv run --project airflow-core pytest airflow-core/tests/unit/timetables/test_interval_timetable.py -q

Copy link
Copy Markdown
Member

@ashb ashb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change seems too simple, it'll need some careful thinking about to make sure there aren't any edge cases

@choo121600 choo121600 added the ready for maintainer review Set after triaging when all criteria pass. label May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for maintainer review Set after triaging when all criteria pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No more dagruns after changing schedule interval

3 participants