kaxil commented on pull request #13932: URL: https://github.com/apache/airflow/pull/13932#issuecomment-768532001
This works but feels ugly and prone to error — i.e if someone adds another `self.dagbag.get `somewhere and misses the `load_op_links` flag — then op link will be loaded again. https://github.com/apache/airflow/pull/13931 looks much cleaner in my opinion -- however we manage some sort of global state -- but we remove the env var before we exist `_do_scheduling` cc @ashb -- I will add tests once we decide on the approach ---------------------------------------------------------------- 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]
