liuandy commented on a change in pull request #9843:
URL: https://github.com/apache/airflow/pull/9843#discussion_r455425890



##########
File path: airflow/models/connection.py
##########
@@ -80,6 +80,7 @@
     "postgres": ("airflow.providers.postgres.hooks.postgres.PostgresHook", 
"postgres_conn_id"),
     "presto": ("airflow.providers.presto.hooks.presto.PrestoHook", 
"presto_conn_id"),
     "redis": ("airflow.providers.redis.hooks.redis.RedisHook", 
"redis_conn_id"),
+    "snowflake": ("airflow.providers.snowflake.hooks.snowflake.SnowflakeHook", 
"snowflake_conn_id"),

Review comment:
       @turbaszek Added, but requires updating the MSSQL mapping to ODBC per 
deprecation notice. Since pymssql is deprecated, the existing hook throws an 
import error for the existing MSSQL mapping.




----------------------------------------------------------------
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]


Reply via email to