Skip to content

Fix backfill behavior for existing dag runs#46959

Merged
dstandish merged 5 commits intoapache:mainfrom
astronomer:add-test-for-reprocessing-existing-runs
Feb 23, 2025
Merged

Fix backfill behavior for existing dag runs#46959
dstandish merged 5 commits intoapache:mainfrom
astronomer:add-test-for-reprocessing-existing-runs

Conversation

@dstandish
Copy link
Contributor

@dstandish dstandish commented Feb 21, 2025

There were some issues in the logic that needed to be resolved.

Some of them were unset variables in the exception block, but the main issue was re the need to clear a dag run when the log date is there.

Replaces #46554
Closes #46278

Also resolves comment thread https://github.com/apache/airflow/pull/46248/files#r1952725320.

@dstandish dstandish changed the title Add test for reprocessing existing runs Fix backfill behavior for existing dag runs Feb 21, 2025
@dstandish dstandish marked this pull request as ready for review February 21, 2025 13:02
@dstandish dstandish added the full tests needed We need to run full set of tests for this PR to merge label Feb 21, 2025
There are some issues in the logic that need to be resolved.  I fixed a couple but more work is required
@dstandish dstandish force-pushed the add-test-for-reprocessing-existing-runs branch from 7fa28b3 to 4325379 Compare February 22, 2025 15:29
@dstandish dstandish merged commit 5acc34d into apache:main Feb 23, 2025
90 checks passed
@dstandish dstandish deleted the add-test-for-reprocessing-existing-runs branch February 23, 2025 17:45
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 2025
There were some issues in the logic that needed to be resolved.

Some of them were unset variables in the exception block, but the main issue was re the need to clear a dag run when the log date is there.

Replaces apache#46554
Closes apache#46278

Also resolves comment thread https://github.com/apache/airflow/pull/46248/files#r1952725320.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AIP-83 Amendment refactor reprocess behaviour in backfill

2 participants