Skip to content

fix(serialized_dag): return original DagDependency if Asset Alias has not yet been resolved into asset#49204

Merged
uranusjr merged 3 commits intoapache:mainfrom
astronomer:asset-alias-dep-graph
Apr 15, 2025
Merged

fix(serialized_dag): return original DagDependency if Asset Alias has not yet been resolved into asset#49204
uranusjr merged 3 commits intoapache:mainfrom
astronomer:asset-alias-dep-graph

Conversation

@Lee-W
Copy link
Member

@Lee-W Lee-W commented Apr 14, 2025

Why

When the asset alias cannot be resolved to an actual asset, the dag dependency is not generated.

closes: #49102

What

Generate the original dag dependency object even if the asset alias has not yet been resolved


^ 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 airflow-core/newsfragments.

@Lee-W Lee-W force-pushed the asset-alias-dep-graph branch from bb6f9fd to cb6a580 Compare April 14, 2025 11:23
@Lee-W Lee-W requested a review from bolkedebruin as a code owner April 14, 2025 11:57
@Lee-W Lee-W force-pushed the asset-alias-dep-graph branch from e8694fe to 1eb110c Compare April 14, 2025 13:56
@Lee-W Lee-W force-pushed the asset-alias-dep-graph branch 2 times, most recently from d133fae to b26715e Compare April 14, 2025 15:12
@phanikumv phanikumv requested a review from uranusjr April 14, 2025 16:22
@Lee-W Lee-W force-pushed the asset-alias-dep-graph branch from b26715e to e8c8fc8 Compare April 15, 2025 02:04
@uranusjr uranusjr merged commit 88f9218 into apache:main Apr 15, 2025
49 checks passed
@uranusjr uranusjr deleted the asset-alias-dep-graph branch April 15, 2025 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DAG dependencies not showing up for When Asset Alias are used

3 participants