Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AIP-83 amendment -- update backfill #46202

Open
1 task done
dstandish opened this issue Jan 28, 2025 · 2 comments · May be fixed by #46248
Open
1 task done

AIP-83 amendment -- update backfill #46202

dstandish opened this issue Jan 28, 2025 · 2 comments · May be fixed by #46248
Assignees
Labels
area:backfill Specifically for backfill related kind:meta High-level information important to the community

Comments

@dstandish
Copy link
Contributor

dstandish commented Jan 28, 2025

Body

Once we've restored uniqueness on logical date, backfill logic can be made a bit simpler.

We should go through, and in places where we guard against the possibility of multiple runs for a given logical date, we should simplify the logic.

Additionally, in places where we do locking to avoid the creation of multiple concurrent runs for a given logical date, we should just rely on the fact that there is a unique constraint to protect us.

We also need to restore the old (2.x) behavior re existing runs that need to be reprocessed. When the reprocess behavior would result in a new run, we clear the existing run. We don't delete the row but update the fields that need to be updated -- whatever was done in 2.x. At least this means changing the state, but probably also start_date and maybe other fields as well.

(reference: https://cwiki.apache.org/confluence/display/AIRFLOW/Option+2+clarification+doc+WIP)

Committer

  • I acknowledge that I am a maintainer/committer of the Apache Airflow project.
@dstandish dstandish added the kind:meta High-level information important to the community label Jan 28, 2025
@dstandish dstandish moved this to Todo in AIP-83 amendment Jan 28, 2025
@dosubot dosubot bot added the area:backfill Specifically for backfill related label Jan 28, 2025
@prabhusneha
Copy link
Contributor

I can pick this task.

@prabhusneha
Copy link
Contributor

prabhusneha commented Jan 29, 2025

PR : #46248
#46187 needs to be merged before closing this one.

@phanikumv phanikumv moved this from Todo to In Progress in AIP-83 amendment Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:backfill Specifically for backfill related kind:meta High-level information important to the community
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

2 participants