3dbrows commented on PR #13633: URL: https://github.com/apache/arrow/pull/13633#issuecomment-1195701184
Thanks guys. I've updated the code with some illustrative (not final) code of what I think the idea is. Can you confirm this is what you meant? What I'm driving at is that we have a subclass of `S3RetryStrategy` called `AwsRetryStrategy` which takes a constructor argument of a stock AWS strategy, itself having been created by `GetS3RetryStrategy()`. (It might be better to have this live elsewhere.) We could of course broaden `GetS3RetryStrategy()` to take the name of any strategy, not just an AWS one. You could pass in a string like `aws_standard` or `arrow_connect` or whatever. (Or enum, even better.) After all, it ultimately just returns you an `S3RetryStrategy`. -- 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]
