thesuperzapper commented on issue #36090:
URL: https://github.com/apache/airflow/issues/36090#issuecomment-2101913601

   @sunank200 I see you have removed the check for a RUNNING state, and now 
only check if it's not in a DEFERRED state, for example:
   
   
https://github.com/apache/airflow/blob/3938f71dfae21c84a3518625543a28ad02edf641/airflow/providers/google/cloud/triggers/dataproc.py#L152
   
   This seems reasonable, and might actually be safer, because it ensures that 
if the TaskInstance has started RUNNING again (perhaps try number 2+ of the 
same task), that external jobs triggered by older tries are killed.


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