ashb commented on pull request #18499:
URL: https://github.com/apache/airflow/pull/18499#issuecomment-926683538


   Hmmmmm, the selectin to the relationship is probably safer, but in cases 
where we _know_ we are going to do this it's better to add 
`options(eagerload(TI.dag_run))` etc to avoid the extra query.
   
   No we can't remove it -- we can deprecate it but it can't be removed until 
3.0


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