o-nikolas commented on PR #26886: URL: https://github.com/apache/airflow/pull/26886#issuecomment-1270220843
> > 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) Yeah, this is a constant balance in Airflow provider packages, the question of "when is a hook/operator adding enough value over just using the underlying service sdk". In this case the user is getting some value from the file/location creation so it's a hard call. -- 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]
