3dbrows commented on PR #13633: URL: https://github.com/apache/arrow/pull/13633#issuecomment-1194320102
No problem. I'm wondering if the best approach here is to ditch `S3RetryStrategy` altogether and re-write `ConnectRetryStrategy` (and others) to subclass `Aws::Client::RetryStrategy`. There might be context I am missing though as to why that is not the approach already taken. (Maybe testability is impaired in some fashion, thinking out loud.) -- 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]
