kszlim commented on issue #5378: URL: https://github.com/apache/arrow-rs/issues/5378#issuecomment-1936798128
Do you think it'd be alright to modify: https://github.com/apache/arrow-rs/blob/master/object_store/src/client/retry.rs#L266 to: `!(e.is_parse() || e.is_parse_status() || e.is_parse_too_large() || e.is_user() || e.is_canceled())` Which would be much more aggressive wrt what states to retry on? Ideally the error `Kind` in hyper would be publically exposed to allow deeper introspection into failure modes that can be retried, but that doesn't seem like it's going to be on the roadmap for a while (see: https://github.com/hyperium/hyper/issues/2845). Whilst this might be more aggressive than desired, I think users could accordingly adjust their retries/backoff config to compensate, what do you think? -- 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]
