TJaniF commented on PR #34392:
URL: https://github.com/apache/airflow/pull/34392#issuecomment-1737849365

   Thank you! Hmm... good point! My thinking was that since currently a 
`failed` upstream task causes the all_skipped-task to end up in `skipped`, then 
`upstream_failed` should do the same but that itself feels a little weird. 
   
   Yeah I agree this:
   task `fail` - task `upstream fail` - all_skipped-task `upstream fail`
   task `fail` - all-skipped-task `upstream fail` 
   task `success` - all-skipped-task `skip`
   
   makes more sense.
   
   I'll make the change :) 


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