ephraimbuddy commented on code in PR #55139:
URL: https://github.com/apache/airflow/pull/55139#discussion_r2322560414
##########
providers/fab/src/airflow/providers/fab/auth_manager/security_manager/override.py:
##########
@@ -126,7 +126,7 @@ def _iter_dags() -> Iterable[DAG]:
with create_session() as session:
yield from DBDagBag().iter_all_latest_version_dags(session=session)
else:
- from airflow.models.dagbag import DagBag
+ from airflow.models import DagBag
Review Comment:
Above is DBDagBag. Below is DagBag. However, importing from airflow.models
is same as airflow.models.dagbag. I think people don't know they can import
directly from models. That's my understanding. Do you prefer
airflow.dag_processing.dagbag?
--
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]
