amoghrajesh commented on PR #66160:
URL: https://github.com/apache/airflow/pull/66160#issuecomment-4388200768

   @Lee-W I tested it and confirmed `asset_state` was absent from context 
entirely when the only inlet is `AssetAlias` as I had made note in the PR desc 
and code comments which i was going to handle later but now that you mentioned 
it, I am fixing it now. `AssetAlias` inlets are resolved to their concrete 
assets at context build time via a new `GetAssetsByAlias` comms message, so 
`context["asset_state"][Asset(name="a")]` works when the alias resolves to that 
asset. For an alias that maps to nothing, `asset_state` is still in context but 
empty. Returning 200 [] rather than 404 for unknown aliases since it is a 
collection rest API endpoint.
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to