Bisk1 commented on PR #33746:
URL: https://github.com/apache/airflow/pull/33746#issuecomment-1693902908

   I found out about it when I was troubleshooting another issue. A task state 
is never set to SHUTDOWN except for tests. I think originally this state was 
supposed to handle tasks that were running but were externally cleared (what is 
currently handled by RESTARTING).
   
   Apparently the author of that change 
(https://github.com/apache/airflow/pull/16681) mistakenly assumed that this 
state is used for tasks externally marked as failed. In reality, when marking a 
task externally as failed, webserver sets it directly to FAILED state, as can 
be verified by code inspection.


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