You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "1inuxoid (via GitHub)" <gi...@apache.org> on 2023/02/09 20:48:44 UTC

[GitHub] [airflow] 1inuxoid commented on issue #29422: Multiple AWS connections support in DynamoDBToS3Operator

1inuxoid commented on issue #29422:
URL: https://github.com/apache/airflow/issues/29422#issuecomment-1424815215

   @eladkal, I can see that `RedshiftToS3Operator` and `S3ToRedshiftOperator` already implement a similar pattern using `redshift_conn_id` connection argument.
   I don't see any other operators in this "transfer family".
   
   Also, I think the semantics of two AWS connections in case of such `AwsToAwsBaseTransferOperator` would be tricky and not straightforward at all. How would you even name them if they are not service-specific? 
   
   I'll probably continue with a change to `DynamoDBToS3Operator` specifically for now, however, I'm open to suggestions on how to address this semantical challenge. 


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