Lee-W commented on PR #44831:
URL: https://github.com/apache/airflow/pull/44831#issuecomment-2540956199

   > Should we also fail if an inactive asset is added by the user to an asset 
alias? I feel we should for consistency.
   
   Yes, this is also missed.
   
   > If that’s the case, this should better be done _after_ the task is run 
(during the event-pushing phase) instead.
   
   I'm not sure 🤔 If we already know the task should fail, I think we should 
fail it at earlier stage (i.e., inlets and outlets cases). If this is a super 
resource consumsing task, this helps saving resource. But for the asset alias 
case, this is not something we can handle before task execution, we should do 
it after task execution.


-- 
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