dstandish commented on PR #24613: URL: https://github.com/apache/airflow/pull/24613#issuecomment-1165540629
> We put `DAG` and `DagModel` in the same module, so the same can be done here. yeah i know about dagmodel @uranusjr, but @ashb's comment indicated to me that putting in diff location was one motivation for using a separate class: > Main reason we might want to avoid using this: Loading SQLA and all of the models is slow (Airflow does this right now at "boot" anyway, but I am working on making that not be the case) The import path (airflow.models.dataset) though we can fix that with a lazy import in airflow/__init__.py like we do DAG and XComArg. but i might be misunderstanding that -- 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]
