ashb commented on a change in pull request #15969:
URL: https://github.com/apache/airflow/pull/15969#discussion_r636141716
##########
File path: airflow/configuration.py
##########
@@ -97,8 +98,6 @@ def default_config_yaml() -> dict:
:return: Python dictionary containing configs & their info
"""
- import airflow.utils.yaml as yaml
Review comment:
This was intentional here and not at the module level -- importing cyaml
is "relatively" slow, and I wanted to delay this unless its _needed_ as
`airflow.configuration` is in the critical path right now when importing
`airflow` (yes, it shouldn't be.), and this slows down all CLI commands.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
