You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2019/10/25 16:15:25 UTC

[GitHub] [airflow] mik-laj edited a comment on issue #6376: [WIP] [AIRFLOW-5705] Add creds backend classes including AWS SSM

mik-laj edited a comment on issue #6376: [WIP] [AIRFLOW-5705] Add creds backend classes including AWS SSM
URL: https://github.com/apache/airflow/pull/6376#issuecomment-546409315
 
 
   Currently, PostgresSQL and MySQL hooks do not support this syntax. If we want we can add such parameters in the constructor, but we should do it before accepting this change. You should also look for others to refer to the connection in the remaining code to avoid similar problems. 
   
   We might want to separate the Connection object from the Connection database entity. This object is an SQLAlchemy object, but very often it will be used without SQL Alchemy. This can be confusing.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services