Taragolis commented on PR #26886:
URL: https://github.com/apache/airflow/pull/26886#issuecomment-1269801626

   IMHO, I thought that this method create more problem rather than solve (also 
have issue with SSE Encryption https://github.com/apache/airflow/pull/25835), 
and seem like it main purpose for use in code PythonOperator (and so on), only 
one operator use this method - 
[S3ToMySqlOperator](https://github.com/apache/airflow/blob/06acf40a4337759797f666d5bb27a5a393b74fed/airflow/providers/mysql/transfers/s3_to_mysql.py#L80-L81).
   
   Also it have the same name as 
[boto3.client("s3").download_file](https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/s3.html#S3.Client.download_file)
 but provide different input arguments, and all other community operators use 
this boto3 method by `S3Hook(...).get_conn().download_file(...)` or same method 
from high-level S3 Resource


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to