tustvold commented on code in PR #6158:
URL: https://github.com/apache/arrow-rs/pull/6158#discussion_r1699117288
##########
object_store/src/client/retry.rs:
##########
@@ -30,21 +30,32 @@ use tracing::info;
/// Retry request error
#[derive(Debug, Snafu)]
pub enum Error {
+ /// The response sent by the cloud provider was a redirect without a
Review Comment:
> Are you suggesting something like
Yes, this is also what the underlying HTTP clients do as well. My
aprehension is we're exposing a fair amount of internal details around how we
perform requests, retries, etc... that seems undesirable.
non-exhaustive would help for adding new variants, but we'd be unable to
change existing. If say we wanted to add more context to a retry error message,
this would be a breaking change
--
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]
