dstandish opened a new pull request #20658:
URL: https://github.com/apache/airflow/pull/20658


   The queue's purpose is to track triggers that need to be canceled. I found 
the language `to_delete` a bit confusing because for one it does not actually 
delete them but cancel them.  The deletion work is actually in 
`cleanup_finished_triggers`.  It seems that the canceling of triggers with this 
method will usually not do anything and it's only for cancelling triggers that 
are currently running  but for whatever reason no longer should be.  E.g. when 
a task is killed and therefore the trigger is no longer needed, or some 
multi-triggerer scenarios.
   
   @kaxil i think these methods should be renamed but just not sure if this is 
considered "breaking" since maybe you could say the triggerer process is 
"internal"
   
   


-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to