potiuk commented on PR #55894: URL: https://github.com/apache/airflow/pull/55894#issuecomment-3409977179
> @potiuk What would be the difference? sys.path will get mangled anyway (if used in non-forked env). Are those 3 cases handled in this PR this kind of "fire and forget the environment" situation? Yes. All 3. Which means that if we specifically use it in those 3 places, the "sys.path" mangling is only possible there - rather than in random places where DagBag is used - accidentally. Again - if I am not mistaken about intention, the thing is that DagBag is used in so many places (for example in tests where interpreter is used by multiple tests) that adding a potentially breaking parameter. It's purely "prevent accidental errors" difference. -- 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]
