Skip to content

Use name passed to @asset decorator when fetching the asset.#56434

Merged
Lee-W merged 2 commits intoapache:mainfrom
tirkarthi:gh56401
Oct 14, 2025
Merged

Use name passed to @asset decorator when fetching the asset.#56434
Lee-W merged 2 commits intoapache:mainfrom
tirkarthi:gh56401

Conversation

@tirkarthi
Copy link
Copy Markdown
Contributor

When name is used in @asset decorator then fetching asset should use the value instead of always using the function name which leads to asset not being found.

closes #56401

@tirkarthi tirkarthi requested review from Lee-W and uranusjr October 6, 2025 14:32
@ashb ashb changed the title Use name when passed to asset decorator for fetching the asset. Use name passed to @asset decorator when fetching the asset. Oct 6, 2025
Comment thread task-sdk/src/airflow/sdk/definitions/asset/decorators.py Outdated
@Lee-W Lee-W merged commit 1cbed3d into apache:main Oct 14, 2025
81 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 14, 2025
…sset. (#56434)

(cherry picked from commit 1cbed3d)

Co-authored-by: Karthikeyan Singaravelan <tir.karthi@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

Backport successfully created: v3-1-test

Status Branch Result
v3-1-test PR Link

github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Oct 14, 2025
…sset. (apache#56434)

(cherry picked from commit 1cbed3d)

Co-authored-by: Karthikeyan Singaravelan <tir.karthi@gmail.com>
Lee-W pushed a commit that referenced this pull request Oct 14, 2025
…sset. (#56434) (#56611)

Co-authored-by: Karthikeyan Singaravelan <tir.karthi@gmail.com>
TyrellHaywood pushed a commit to TyrellHaywood/airflow that referenced this pull request Oct 22, 2025
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.

ASSET_NOT_FOUND when passing name argument to @asset

4 participants