ashb commented on pull request #9170: URL: https://github.com/apache/airflow/pull/9170#issuecomment-641910253
> It looks good, but I don't know if it suits our code style. We use explicit joins instead of relying on ORM generated more often. @mik-laj I agree we don't _currently_ use relationships; I guess I am proposing that we currently start using them. The queries generated is the same/similar. Using explicit relationships feels like a good idea to me, as it documents "yes, this is an 'official' relationship between these two tables, not just something that happens to have the same named columns". ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
