dstandish commented on PR #24743:
URL: https://github.com/apache/airflow/pull/24743#issuecomment-1175338845

   > The default behaviour, and the only behaviour in this AIP would be "any 
changed"
   
   Roger yeah I did not see that line.  To me "all changed" feels like the more 
natural default behavior, and that's what would get my vote.  But let's 
discuss.  I think that ultimately if implementing "all changed" this row lock 
may not be what we need.  And there are details concerning dagrun creation 
behavior that we should think through in any case.  E.g. with "any changed", 
even with this row lock strategy, if you have 10 upstream datasets, and each is 
in its own dag, you'll get 10 dag runs created.  But if you have 10 upstream 
datasets and they are in the same dag, you'll get 1 dag run created.


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