[GitHub] [airflow] subkanthi commented on issue #19489: PostgresSqlHook needs to override DbApiHook.get_uri to pull in extra for client_encoding=utf-8 during create_engine

[GitBox]

subkanthi commented on issue #19489:
URL: https://github.com/apache/airflow/issues/19489#issuecomment-964356546


   Just copying the same get_uri to postgres.py seems to like duplicate logic, 
should we have a common base class for both postgres and mysql  instead of 
deriving from DbApiHook?


-- 
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-unsubscr...@airflow.apache.org

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