ashb commented on a change in pull request #17304:
URL: https://github.com/apache/airflow/pull/17304#discussion_r679008413
##########
File path: airflow/__init__.py
##########
@@ -74,11 +74,13 @@ def __getattr__(name):
if not settings.LAZY_LOAD_PROVIDERS:
from airflow import providers_manager
- providers_manager.ProvidersManager().initialize_providers_manager()
+ manager = providers_manager.ProvidersManager()
+ manager.initialize_providers_list()
Review comment:
I disagree -- the intent here is to preload the hooks and links -- that
we need to load the list to do that is to me an implementation detail that we
don't strictly care about.
But I don't feel all that strongly about it.
--
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]
