Skip to content

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

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

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 #46584
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 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
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 13, 2025
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 13, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 13, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 13, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 13, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py

# Conflicts:
#	newsfragments/aip-72.significant.rst
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 13, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py

# Conflicts:
#	newsfragments/aip-72.significant.rst
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 14, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py

# Conflicts:
#	newsfragments/aip-72.significant.rst
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 14, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py

# Conflicts:
#	newsfragments/aip-72.significant.rst
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 14, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py

# Conflicts:
#	newsfragments/aip-72.significant.rst
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 14, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py

# Conflicts:
#	newsfragments/aip-72.significant.rst
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 14, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py

# Conflicts:
#	newsfragments/aip-72.significant.rst
kaxil pushed a commit to shahar1/airflow that referenced this pull request Mar 14, 2025
closes: apache#45823
related: apache#44912

# Conflicts:
#	task-sdk/src/airflow/sdk/execution_time/task_runner.py

# Conflicts:
#	newsfragments/aip-72.significant.rst
kaxil added a commit to shahar1/airflow that referenced this pull request Mar 14, 2025
closes: apache#45823
related: apache#44912
Co-Authored-By: Kaxil Naik <kaxilnaik@gmail.com>
kaxil added a commit that referenced this pull request Mar 14, 2025
closes: #45823
related: #44912

Co-Authored-By: Kaxil Naik <kaxilnaik@gmail.com>
---------

Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
agupta01 pushed a commit to agupta01/airflow that referenced this pull request Mar 21, 2025
closes: apache#45823
related: apache#44912

Co-Authored-By: Kaxil Naik <kaxilnaik@gmail.com>
---------

Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 2025
closes: apache#45823
related: apache#44912

Co-Authored-By: Kaxil Naik <kaxilnaik@gmail.com>
---------

Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy