potiuk commented on PR #30367:
URL: https://github.com/apache/airflow/pull/30367#issuecomment-1490204232
Are you sure there are no side effects? Can you take a look at initialize
and possibly add a test or two showing that things are working as expected.
There is a VERY strong comment in the code:
```
# Perform side-effects unless someone has explicitly opted out before import
# WARNING: DO NOT USE THIS UNLESS YOU REALLY KNOW WHAT YOU'RE DOING.
if not os.environ.get("_AIRFLOW__AS_LIBRARY", None):
settings.initialize()
```
And I think only looking at what `settings.initialize` does and adding some
related tests can prove that you actually REALLY KNOW WHAT YOU'RE DOING .
Changing production behaviour in order to make "DEBUG" settings behave wrongly
is generally a recipe for disaster unless you REALLY know what you are doing.
--
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]
