You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "jackye1995 (via GitHub)" <gi...@apache.org> on 2026/01/23 18:46:21 UTC

[PR] fix: missing 5xx error body when retry exhausted [arrow-rs-object-store]

jackye1995 opened a new pull request, #618:
URL: https://github.com/apache/arrow-rs-object-store/pull/618

   # Which issue does this PR close?
   
   Closes #617 
   
   # Rationale for this change
    
   Currently we only capture the error body after retry exhausted for client 4xx error. This makes it unclear about the exact S3 error message when the error is a server 5xx error.
   
   # What changes are included in this PR?
   
   Make sure we always capture the error response body after retry exhausted.
   
   # Are there any user-facing changes?
   
   No user facing or breaking changes
   


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: missing 5xx error body when retry exhausted [arrow-rs-object-store]

Posted by "crepererum (via GitHub)" <gi...@apache.org>.
crepererum commented on PR #618:
URL: https://github.com/apache/arrow-rs-object-store/pull/618#issuecomment-3853590477

   CI got fixed in the meantime, can you please rebase?


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: missing 5xx error body when retry exhausted [arrow-rs-object-store]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on PR #618:
URL: https://github.com/apache/arrow-rs-object-store/pull/618#issuecomment-4045922837

   Hi @jackye1995  -- we are preparing for another object store release next week and would like to include this PR. I merged up from main to try and get a clean CI run


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix: missing 5xx error body when retry exhausted [arrow-rs-object-store]

Posted by "crepererum (via GitHub)" <gi...@apache.org>.
crepererum merged PR #618:
URL: https://github.com/apache/arrow-rs-object-store/pull/618


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org