wjones127 commented on PR #13633: URL: https://github.com/apache/arrow/pull/13633#issuecomment-1194196307
Hmm I think I may have led you astray. Sorry about that. > My confusion is that it feels like attempting duck-typing in a type-safe language, which I didn't think would be something you could do. I tested this and your intuition is correct. Odd though that we don't provide a way to wrap an `Aws::Client::RetryStrategy` in a `arrow::fs::AwsRetryStrategy`. Maybe it's just that no one has requested that. @pitrou @lidavidm any thoughts on that? Having looked a little more at boto3, I think we probably want to support two parameters: `retry_mode` and `retry_attempts`, corresponding to the boto3 arguments here: https://boto3.amazonaws.com/v1/documentation/api/latest/guide/retries.html Then we'll need a C++ function: ```cpp arrow::fs::AwsRetryStrategy GetS3RetryStrategy(const std::string& name, long retry_attempts) { ... } ``` You should then be able to expose that function in Cython. One issue: it does look like which retry strategies are available depends on the version of the AwsSdk. Adaptive isn't available in the one currently installed on my computer. -- 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]
