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 2021/12/22 16:34:18 UTC

[GitHub] [airflow] aa3pankaj edited a comment on pull request #20459: Adding snowflake_to_s3 transfer operator, Updates in s3_to_snowflake

aa3pankaj edited a comment on pull request #20459:
URL: https://github.com/apache/airflow/pull/20459#issuecomment-999709969


   > Why just SnowflakeOperator is not enough? The operator logic is quite simple and I think we can expect the user to just copy the SQL query from the Snowflake documentation.
   
   @mik-laj We need this for the same reason why we have the SnowflakeOperator when we can use SnowflakeHook directly to run queries.
   
   And S3ToSnowflake related changes are required since we already have this operator merged, and currently query_ids and execution_info not exposed from this operator.
   


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

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