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.

Sign up for GitHub

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

Jump to bottom

Refactor dedent nested loops #34409

Merged
merged 2 commits into from Sep 27, 2023
Merged

Refactor dedent nested loops #34409

merged 2 commits into from Sep 27, 2023

Conversation

eumiro
Copy link
Contributor

@eumiro eumiro commented Sep 15, 2023

Using itertools.product only where the two nested loops are equivalent and independent.

@boring-cyborg boring-cyborg bot added area:core-operators Operators, Sensors and hooks within Core Airflow area:dev-tools area:providers area:webserver Webserver related Issues kind:documentation provider:google Google (including GCP) related issues labels Sep 15, 2023
@eumiro eumiro marked this pull request as ready for review September 17, 2023 16:45
@vincbeck vincbeck merged commit 07fe1d2 into apache:main Sep 27, 2023
65 checks passed
@eumiro eumiro deleted the dedent-nested-loops branch September 27, 2023 16:53
@ephraimbuddy ephraimbuddy added this to the Airflow 2.7.2 milestone Oct 3, 2023
@ephraimbuddy ephraimbuddy added the type:misc/internal Changelog: Misc changes that should appear in change log label Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@uranusjr uranusjr uranusjr approved these changes

@vincbeck vincbeck vincbeck approved these changes

@potiuk potiuk Awaiting requested review from potiuk potiuk is a code owner

@ashb ashb Awaiting requested review from ashb ashb is a code owner

@jedcunningham jedcunningham Awaiting requested review from jedcunningham jedcunningham is a code owner

@ryanahamilton ryanahamilton Awaiting requested review from ryanahamilton ryanahamilton is a code owner

@bbovenzi bbovenzi Awaiting requested review from bbovenzi bbovenzi is a code owner

@pierrejeambrun pierrejeambrun Awaiting requested review from pierrejeambrun pierrejeambrun is a code owner

@kaxil kaxil Awaiting requested review from kaxil kaxil is a code owner

@XD-DENG XD-DENG Awaiting requested review from XD-DENG XD-DENG is a code owner

Assignees
No one assigned
Labels
area:core-operators Operators, Sensors and hooks within Core Airflow area:dev-tools area:providers area:webserver Webserver related Issues kind:documentation provider:google Google (including GCP) related issues type:misc/internal Changelog: Misc changes that should appear in change log
Projects
None yet
Milestone
Airflow 2.8.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@eumiro @uranusjr @vincbeck @ephraimbuddy