jedcunningham commented on code in PR #30934:
URL: https://github.com/apache/airflow/pull/30934#discussion_r1179808652
##########
airflow/utils/cli_app_builder.py:
##########
@@ -24,12 +24,15 @@
from flask import Flask
from airflow.www.extensions.init_appbuilder import AirflowAppBuilder,
init_appbuilder
+from airflow.www.extensions.init_views import init_plugins
@lru_cache(maxsize=None)
def _return_appbuilder(app: Flask) -> AirflowAppBuilder:
"""Returns an appbuilder instance for the given app"""
- return init_appbuilder(app)
+ init_appbuilder(app)
+ init_plugins(app)
Review Comment:
Doing it this way instead of just reverting sync-perm to using the full app
saves us about .75 seconds. I wasn't able to reproduce the 3-5x gain in the
original pr 🤷♂️.
--
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]
