Skip to content

Ensure that we don't schedule all mapped TIs when one is cleared#23130

Merged
ashb merged 1 commit into
apache:mainfrom
astronomer:only-schedule-some-mapped-tis
Apr 21, 2022
Merged

Ensure that we don't schedule all mapped TIs when one is cleared#23130
ashb merged 1 commit into
apache:mainfrom
astronomer:only-schedule-some-mapped-tis

Conversation

@ashb
Copy link
Copy Markdown
Member

@ashb ashb commented Apr 20, 2022

We had missed a query and when setting TIs to scheduled state we were mistakenly not filtering on map_index, meaning that if you cleared a single mapped TI it would re-run all of them for that dagrun


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragement file, named {pr_number}.significant.rst, in newsfragments.

We had missed a query and when setting TIs to scheduled state we were
mistakenly not filtering on map_index, meaning that if you cleared a
single mapped TI it would re-run _all_ of them for that dagrun
@ashb ashb requested review from XD-DENG and kaxil as code owners April 20, 2022 20:31
@ashb ashb requested a review from bbovenzi April 20, 2022 20:31
@ashb
Copy link
Copy Markdown
Member Author

ashb commented Apr 20, 2022

(This came about because of adding #23127)

@ashb ashb merged commit af45483 into apache:main Apr 21, 2022
@ashb ashb deleted the only-schedule-some-mapped-tis branch April 21, 2022 12:04
@jedcunningham jedcunningham added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Apr 25, 2022
@jedcunningham jedcunningham added this to the Airflow 2.3.0 milestone Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dynamic-task-mapping AIP-42 changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants