Skip to content

TS SDK: Collect dag_dependencies in the tasks pass - #75032

Merged
vatsrahul1001 merged 1 commit into
apache:mainfrom
guan404ming:ts-sdk-dag-deps-one-pass
Oct 20, 2026
Merged

TS SDK: Collect dag_dependencies in the tasks pass#75032
vatsrahul1001 merged 1 commit into
apache:mainfrom
guan404ming:ts-sdk-dag-deps-one-pass

Conversation

@guan404ming

@guan404ming guan404ming commented Oct 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Related

Why

  • Serializing a Dag looped over every task a second time only to collect Dag dependencies.

How

  • Collect dag_dependencies in the existing tasks.map() pass instead of a second loop over the tasks.
Was generative AI tooling used to co-author this PR?

@guan404ming
guan404ming marked this pull request as ready for review October 19, 2026 05:14
@jason810496
jason810496 force-pushed the ts-sdk-dag-deps-one-pass branch from 98f6333 to cf80e87 Compare October 20, 2026 06:08
@jason810496 jason810496 added this to the Airflow 3.4.0 milestone Oct 20, 2026
@jason810496 jason810496 added the backport-to-v3-4-test Backport to v3-4-test label Oct 20, 2026
jason810496
jason810496 approved these changes Oct 20, 2026
View reviewed changes

@jason810496 jason810496 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the follow-up.

@vatsrahul1001
vatsrahul1001 merged commit 94ce2c9 into apache:main Oct 20, 2026
78 checks passed
@github-actions

github-actions Bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-4-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
✅ v3-4-test PR Link

vatsrahul1001 pushed a commit that referenced this pull request Oct 20, 2026
@github-actions @guan404ming
[v3-4-test] TS SDK: Collect dag_dependencies in the tasks pass (#75032)…
fb0138b 
… (#75191)

(cherry picked from commit 94ce2c9)

Co-authored-by: Guan-Ming Chiu <105915352+guan404ming@users.noreply.github.com>
@guan404ming

guan404ming commented Oct 20, 2026

Copy link
Copy Markdown
Member Author

Thanks!

@guan404ming
guan404ming deleted the ts-sdk-dag-deps-one-pass branch October 20, 2026 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jason810496 jason810496 jason810496 approved these changes

Assignees

No one assigned

Labels

area:ts-sdk backport-to-v3-4-test Backport to v3-4-test

Projects

None yet

Milestone

Airflow 3.4.0

Development

Successfully merging this pull request may close these issues.

3 participants

@guan404ming @jason810496 @vatsrahul1001