You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "uranusjr (via GitHub)" <gi...@apache.org> on 2025/09/03 02:30:16 UTC
[PR] Revert "Support templated extra in outlets assets (#54885)" [airflow]
uranusjr opened a new pull request, #55199:
URL: https://github.com/apache/airflow/pull/55199
This reverts #54885. The PR should not have gone in. Asset extras and asset event extras are different entities, and NOT DESIGNED TO BE MERGED.
https://airflow.apache.org/docs/apache-airflow/stable/authoring-and-scheduling/assets.html#attaching-extra-information-to-an-emitting-asset-event
This is an explicit design. It can be changed, but should not be done in a simple PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Revert "Support templated extra in outlets assets (#54885)" [airflow]
Posted by "hussein-awala (via GitHub)" <gi...@apache.org>.
hussein-awala merged PR #55199:
URL: https://github.com/apache/airflow/pull/55199
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Revert "Support templated extra in outlets assets (#54885)" [airflow]
Posted by "uranusjr (via GitHub)" <gi...@apache.org>.
uranusjr commented on PR #55199:
URL: https://github.com/apache/airflow/pull/55199#issuecomment-3247840018
See #55200 for a proposal to improve the situation.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Revert "Support templated extra in outlets assets (#54885)" [airflow]
Posted by "Lee-W (via GitHub)" <gi...@apache.org>.
Lee-W commented on PR #55199:
URL: https://github.com/apache/airflow/pull/55199#issuecomment-3247492100
These two interface looks almost the same. Should we remove or rename one of them to avoid this kind of misunderstanding? I think AssetEvent extra is more useful than the asset one but asset extra also might have some use cases
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org