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

Rewrite recursion into iteration #25898

Merged
merged 2 commits into from
Aug 23, 2022
Merged

Conversation

uranusjr
Copy link
Member

This avoids a RecursionError when parsing a DAG with very deep dependencies that takes the interpreter over sys.getrecursionlimit().

This avoids a RecursionError when parsing a DAG with very deep
dependencies that takes the interpreter over sys.getrecursionlimit().
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yeah !

@kaxil kaxil merged commit b92f619 into apache:main Aug 23, 2022
@kaxil kaxil deleted the dag-parsing-use-loop branch August 23, 2022 11:40
@kaxil kaxil added this to the Airflow 2.3.4 milestone Aug 23, 2022
@ashb ashb modified the milestones: Airflow 2.3.4, Airflow 2.4.0 Aug 23, 2022
@jedcunningham jedcunningham added the type:bug-fix Changelog: Bug Fixes label Sep 12, 2022
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Jan 30, 2023
Cherry-picked changes:
Rewrite recursion into iteration
apache/airflow#26175
Rewrite recursion into iteration
apache/airflow#25898

Change-Id: I529f90aae79ad2716b9282e9b70db91a4f64d9f7
GitOrigin-RevId: dae6b05f8266ccff97e1a565e36a5f580db4c79d
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Jan 30, 2023
Cherry-picked changes:
Rewrite recursion into iteration
apache/airflow#26175
Rewrite recursion into iteration
apache/airflow#25898

Change-Id: I3d886329b1d856c4b276d265dd7750dae2b200ba
GitOrigin-RevId: 37b365dc035d3e23961683e322b214fb937fab99
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants