Repository navigation
Conversation
bc7581a to
ec8660c
Compare
ec8660c to
0177dd1
Compare
2cab1bb to
3f81dac
Compare
3f81dac to
636adac
Compare
|
@yuseok89 — I've removed the Automated triage note drafted by an AI-assisted tool — may get things wrong; a real Apache Airflow maintainer takes the next look once it's green. (why automated) Drafted-by: Claude Code (Opus 4.8); reviewed by @potiuk before posting |
016ad8c to
6bd844f
Compare
6bd844f to
957eb15
Compare
824a172 to
562418a
Compare
fb20e44 to
2d7a20a
Compare
2d7a20a to
6d68027
Compare
# Conflicts: # airflow-core/docs/migrations-ref.rst # airflow-core/src/airflow/utils/db.py
|
Hello @yuseok89 - thank you for your contributions to Apache Airflow! The Airflow community has introduced a limit of 5 open pull requests at a time for contributors without write access to the repository. You currently have 11 open pull requests, so - as a one-time step of introducing the limit - we closed the ones where maintainers have not engaged yet:
These pull requests stay open because maintainers are already engaged in them - they count towards your limit:
This is not a judgement of you or of your changes. We never told contributors before that opening many pull requests at once was a problem, so there is nothing to feel bad about - and nothing is lost: your branches, commits and the review history stay where they are. What we ask you to do is to make your first prioritization decision: choose which of the pull requests above matter most to you, and reopen them (up to 5 open at a time, including the ones still open) with the "Reopen pull request" button or While your pull requests are waiting for review, the most valuable thing you can do is help in other ways - reviewing other contributors' pull requests, helping with issues, and taking part in the discussions on the devlist and Slack. Why we introduced the limit, what it means for you and how to reopen or restore a pull request is explained in https://github.com/apache/airflow/blob/main/contributing-docs/32_open_pull_request_limit.rst. Drafted-by: Claude Code (Opus 5); reviewed by @potiuk before posting |
related: #21867
Implements the minimal v1 scope for TaskGroup retries I proposed on the dev list, a first step toward #21867. A
TaskGroupcan now be retried as a unit, the waySubDagonce could. The richer options from the issue and thread are left for follow-up PRs.What's in this v1
Group retries run at the top of DagRun.update_state, before scheduling decisions, so a failure is cleared before it can propagate downstream. Covered by SDK, serialization, and scheduler integration tests.
Out of v1 (follow-ups)
retry_condition (any/last/custom), retry_strategy (all / only-failed), group-level retry_delay, mapped-group retries.
Demo
The example DAG has a
collect_and_publishgroup withretries=2, andpublishis intentionally made to fail on its first two attempts and succeed on the third, so the group is retried twice before it goes green. In the run below:collectandpublishboth run 3 times. A group retry re-runs every task in the group, not just the one that failed.startandfinish, which sit outside the group, each run once.Screen.Recording.2026-06-06.at.11.47.22.PM.mov
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.Important
🛠️ Maintainer triage note for @yuseok89 · by
@potiuk· 2026-07-18 16:36 UTCI've removed the
ready for maintainer reviewlabel — the next step here is yours:main. See docs.It'll return to the maintainer queue automatically once resolved — no need to re-add the label by hand. See the Pull Request quality criteria for details.
The ball is in your court — you've been assigned. Rebase onto
main, push, then mark it Ready for review.Automated triage — may be imperfect; a maintainer takes the next look.