jason810496 commented on code in PR #62696:
URL: https://github.com/apache/airflow/pull/62696#discussion_r2972684761
##########
shared/configuration/src/airflow_shared/configuration/parser.py:
##########
@@ -1046,6 +1172,49 @@ def _resolve_deprecated_lookup(
return section, key, deprecated_section, deprecated_key,
warning_emitted
+ def load_providers_configuration(self) -> None:
+ """
+ Load configuration for providers.
+
+ This should be done after initial configuration have been performed.
Initializing and discovering
+ providers is an expensive operation and cannot be performed when we
load configuration for the first
+ time when airflow starts, because we initialize configuration very
early, during importing of the
+ `airflow` package and the module is not yet ready to be used when it
happens and until configuration
+ and settings are loaded. Therefore, in order to reload provider
configuration we need to additionally
+ load provider - specific configuration.
+ """
+ log.debug("Loading providers configuration")
+
+ self.restore_core_default_configuration()
Review Comment:
Moved `restore_core_default_configuration` to shared lib level.
--
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]
