potiuk commented on code in PR #40916:
URL: https://github.com/apache/airflow/pull/40916#discussion_r1686886717
##########
airflow/cli/commands/internal_api_command.py:
##########
@@ -222,7 +222,8 @@ def create_app(config=None, testing=False):
if "SQLALCHEMY_ENGINE_OPTIONS" not in flask_app.config:
flask_app.config["SQLALCHEMY_ENGINE_OPTIONS"] =
settings.prepare_engine_args()
- InternalApiConfig.force_database_direct_access("Gunicorn worker
initialization")
+ if conf.getboolean("core", "database_access_isolation", fallback=False):
+ InternalApiConfig.set_use_database_access("Gunicorn worker
initialization")
Review Comment:
Yes. Why not. It would continue to work, byt it's more consistent this way.
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
