shubhamraj-git commented on code in PR #45139:
URL: https://github.com/apache/airflow/pull/45139#discussion_r1894861522
##########
providers/src/airflow/providers/fab/auth_manager/cli_commands/utils.py:
##########
@@ -50,4 +54,12 @@ def get_application_builder() ->
Generator[AirflowAppBuilder, None, None]:
with flask_app.app_context():
# Enable customizations in webserver_config.py to be applied via
Flask.current_app.
flask_app.config.from_pyfile(webserver_config, silent=True)
+ flask_app.config["SQLALCHEMY_DATABASE_URI"] = conf.get("database",
"SQL_ALCHEMY_CONN")
+ url = make_url(flask_app.config["SQLALCHEMY_DATABASE_URI"])
+ if url.drivername == "sqlite" and url.database and not
isabs(url.database):
+ raise AirflowConfigException(
+ f'Cannot use relative path: `{conf.get("database",
"SQL_ALCHEMY_CONN")}` to connect to sqlite. '
Review Comment:
This has been taken from already existing code, Double quotes are used
inside the string also, to differentiate this I think they have added the
single quotes instead, the space is left to keep the next line look good. Do
you still want to me to replace them?
--
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]
