You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "hussein-awala (via GitHub)" <gi...@apache.org> on 2024/01/08 23:02:23 UTC

[PR] Provide the logger_name param to base hook in order to override the logger name [airflow]

hussein-awala opened a new pull request, #36674:
URL: https://github.com/apache/airflow/pull/36674

   The BaseHook accepts a parameter `logger_name` to override the logger_name in the LoggingMixin. Unfortunately, some of the hooks don't provide this parameter to the parent class, and the only way to override the logger name is doing it on an extra step and overriding a protected class attribute:
   ```python
   hook = SomeHook(...)
   hook._logger_name = "<some name>"
   ```


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Provide the logger_name param to base hook in order to override the logger name [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #36674:
URL: https://github.com/apache/airflow/pull/36674


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org