Taragolis commented on PR #26886: URL: https://github.com/apache/airflow/pull/26886#issuecomment-1269838838
> This is hook capability which means it can be used by many operators. Check the user issue the use case was to use custom operator with it. Most of the user issue would fix if they avoid to use this method and just switch to method provided by boto3 and do whatever they wanted. Because in this case S3Hook would use only for authentication and creation client. All other stuff they can read in boto3 documentation, rather than also try to understand why it won't work in this method and why it work different rather than [download_file](https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/s3.html#S3.Client.download_file) Again it just my IMHO, but I want to highlight that might this is a reason why other community provided operators in general try to avoid use this method. And another IMHO, better just warn user (UserWarning, docstring, provider documentation) there is another method to achieve everything they wanted in their custom components rather than struggling with this method. -- 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]
