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/01/03 09:25:43 UTC

[GitHub] jgao54 commented on issue #4423: [AIRFLOW-3618] - Fix models.connection.Connection get_hook function

jgao54 commented on issue #4423: [AIRFLOW-3618] - Fix models.connection.Connection get_hook function
URL: https://github.com/apache/incubator-airflow/pull/4423#issuecomment-451090738
 
 
   All the conn type are defined [here](https://github.com/apache/incubator-airflow/blob/master/airflow/models/connection.py#L53-L92) and they are all lower case consistently. We don't use .lower() because these conn types are hard-coded into the connection.py module.
   
   I noticed that in your JIRA it shows `MySQL` instead, perhaps it's a custom created connection type?  

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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