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


   closes: #18392
   
   alternative to #19546
   
   Currently triggers may be queued up as soon as they leave the "running" set 
`triggers`.  This results in duplicate triggers when a trigger has been removed 
from this set but not yet deleted from the database.  We resolve by checking 
against not just running triggers but all triggers known to the triggerer 
instance.
   
   Previously (#19546) I approached this in a different way.  But as I worked 
out how to reproduce the behavior in a test, I obtained a better understanding 
of the conditions, and realized that there was a simpler way.  
   
   Unioning all those sets is maybe a bit ugly, but it's better than a query, I 
think.
   
   cc @andrewgodwin 


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