potiuk commented on a change in pull request #12466:
URL: https://github.com/apache/airflow/pull/12466#discussion_r527276595
##########
File path: airflow/providers_manager.py
##########
@@ -46,42 +45,134 @@ class ProvidersManager:
"""Manages all provider packages."""
def __init__(self):
+ self.__log = logging.getLogger(__name__)
Review comment:
Privacy by design. I switched to the (recommended by many) mode of
programming - only expose what you expect to be used and everything else
private by design. That makes it much more encapsulated and makes it easier to
avoid accidental uses and unintended side effects.
----------------------------------------------------------------
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]
