kaxil commented on pull request #13932: URL: https://github.com/apache/airflow/pull/13932#issuecomment-770047667
> I prefer this approach to the global state of environment variables. Is it necessary to pass `load_op_links` to DagBag's constructor though? > > I'm not super familiar with the how this is used, so it might not make sense, but could it be passed directly to `deserialize_operator` as an argument? Mainly to reduce the places we need to pass the flag otherwise -- https://github.com/apache/airflow/pull/13932/commits/dab163e80bb62d7d8fb43b4adb7654c5b6c3e3ba is my first commit where I tried something similar but its too ugly -- I passed it all the over the place ---------------------------------------------------------------- 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]
