uranusjr commented on pull request #21851:
URL: https://github.com/apache/airflow/pull/21851#issuecomment-1063789890


   Logic looks good to me, but can you change the code to _not_ use 
`DagRun.find()`, and leave the function unchanged? I don’t think this new logic 
is general enough that we should risk breaking backward compatibility. 
(Eventually I feel `DagRun.find()` should probably be removed entirely, but 
before that we would do better to avoid using it in new code.)


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