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 2020/06/29 07:41:09 UTC

[GitHub] [airflow] eladkal commented on issue #9548: Add support for snowflake on the sql_sensor

eladkal commented on issue #9548:
URL: https://github.com/apache/airflow/issues/9548#issuecomment-650989374


   @KumarManoj-S  note that if this works for SqlSensor there is no reason it won't work for SqlOperators
   https://github.com/apache/airflow/blob/1d36b0303b8632fce6de78ca4e782ae26ee06fea/airflow/sensors/sql_sensor.py#L72
   https://github.com/apache/airflow/blob/0b9bf4a285a074bbde270839a90fb53c257340be/airflow/operators/sql.py#L26
    (not sure why two different lists are needed as lists are the same and there shouldn't be a reason why sensor would work on specific db but the operators won't or vice versa)
   Might worth checking that if you can address both


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