potiuk commented on code in PR #46891:
URL: https://github.com/apache/airflow/pull/46891#discussion_r2211848475
##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -124,6 +127,13 @@ def init_flask_plugins(app: FastAPI) -> None:
try:
from airflow.providers.fab.www.app import create_app
except ImportError:
+ if PY313:
+ log.info(
Review Comment:
No. Can't make it into a warning, because you can have providers that have
some FAB-related plugins but you do not have FAB provider and FAB installed.
And it's because - we have such providers ourselves (DatabricksWorkflowPlugin).
If we turn it into warning, and someone wants to use Databricks provider (but
does not have FAB and does not use the plugin) - they will have unsilenceable
warning.
I guess the problem will be solved when that plugin will be converted to
support dynamically both React Plugin and FAB plugin - based on airflow version
- and then likely we can turn it into a warning suggesting upgrading to latest
version of the provider.
--
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]
