Ferdinanddb commented on issue #50915: URL: https://github.com/apache/airflow/issues/50915#issuecomment-3033961422
@valentinDruzhinin I mentioned it in the issue description but another way of implementing this feature could be to do it via the AWS connection object itself, similar to what is [described here](https://airflow.apache.org/docs/apache-airflow-providers-amazon/stable/connections/aws.html#s3-bucket-configurations). This approach has pros and cons imo: - Cons: - It might mean that one could have to define two AWS connection objects instead of one (with and without RequestPay=requester) because the identity might require this option on some buckets but not everywhere. - Pros: - I think the code could be cleaner with this implementation. - No need to change the check in BaseOperator (I doubt that they decide to disable this check just for this). -- 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]
