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

[v2-10-test] Fix short circuit in mapped tasks #44912

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

shahar1
Copy link
Contributor

@shahar1 shahar1 commented Dec 13, 2024

will be ported to main by: #44925
related: #43883


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an 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 newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@boring-cyborg boring-cyborg bot added the area:core-operators Operators, Sensors and hooks within Core Airflow label Dec 13, 2024
@shahar1 shahar1 force-pushed the fix-dtm-short-circuit-v2-10-test branch from 3918d28 to 5c6f3cc Compare December 13, 2024 12:57
@shahar1 shahar1 changed the title [v2-10-test] Fix skipped tasks handling in mapped operators [v2-10-test] Fix short circuit in mapped tasks Dec 13, 2024
@shahar1 shahar1 force-pushed the fix-dtm-short-circuit-v2-10-test branch 2 times, most recently from 7bcac5c to 8e86c3b Compare December 13, 2024 22:27
@shahar1 shahar1 marked this pull request as ready for review December 13, 2024 22:28
@shahar1 shahar1 requested a review from eladkal December 13, 2024 23:08
@shahar1 shahar1 force-pushed the fix-dtm-short-circuit-v2-10-test branch from 8e86c3b to 414bcf1 Compare December 13, 2024 23:19
@shahar1 shahar1 marked this pull request as draft December 14, 2024 09:43
@shahar1 shahar1 force-pushed the fix-dtm-short-circuit-v2-10-test branch from 414bcf1 to 90ac192 Compare December 14, 2024 09:54
@shahar1 shahar1 marked this pull request as ready for review December 14, 2024 09:57
@shahar1 shahar1 force-pushed the fix-dtm-short-circuit-v2-10-test branch from 90ac192 to e0a935d Compare December 14, 2024 10:12
@kaxil kaxil force-pushed the fix-dtm-short-circuit-v2-10-test branch from e0a935d to ab7835a Compare December 17, 2024 15:08
@kaxil
Copy link
Member

kaxil commented Dec 17, 2024

@shahar1 Could you verify your changes again -- I needed to force-push on v2-10-test to sync it back with v2-10-stable.

@shahar1
Copy link
Contributor Author

shahar1 commented Dec 17, 2024

@shahar1 Could you verify your changes again -- I needed to force-push on v2-10-test to sync it back with v2-10-stable.

Verified and it works great :)

@shahar1 shahar1 force-pushed the fix-dtm-short-circuit-v2-10-test branch from 5be965d to d8109dd Compare December 17, 2024 21:11
@shahar1 shahar1 merged commit ec4db3e into apache:v2-10-test Dec 18, 2024
48 checks passed
@shahar1 shahar1 deleted the fix-dtm-short-circuit-v2-10-test branch December 18, 2024 06:32
@utkarsharma2 utkarsharma2 added the type:bug-fix Changelog: Bug Fixes label Jan 28, 2025
@utkarsharma2 utkarsharma2 added this to the Airflow 2.10.5 milestone Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:core-operators Operators, Sensors and hooks within Core Airflow type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants