sunkickr commented on pull request #14724:
URL: https://github.com/apache/airflow/pull/14724#issuecomment-799037407


   > @sunkickr I'm not sure if having AWS stuff in Snowflake connection is good 
idea. In future there can be any XtoSnowflake operator, will we be extending 
the list?
   
   @turbaszek I agree that the AWS stuff shouldn't be in the connection, but 
the S3ToSnowflakeOperator is currently written in a way that requires this(I 
think). I thought rewriting the S3ToSnowflakeOperator was beyond the scope of 
this PR. Other transfers could have users set up a connection for each service 
so we wouldn't need to extend the list.


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