philjb opened a new pull request, #487:
URL: https://github.com/apache/arrow-rs-object-store/pull/487
On a request retry, it logs an info message stating that an error was
encountered and information about the retry process but it hasn't included any
details about the error that is causing the retry. This PR updates the logging
to include the status if it is a server error and the http error kind if a
transport error occurred. While the last error when retries are exhausted is
returned up the call stack, the intermediate errors need not be exactly the
same. It is helpful to include some minimum information about what error
triggered a retry each time it happens.
# Which issue does this PR close?
<!--
We generally require a GitHub issue to be filed for all bug fixes and
enhancements and this helps us generate change logs for our releases. You can
link an issue to this PR using the GitHub syntax. For example `Closes #123`
indicates that this PR will close issue #123.
-->
Closes #486
# Rationale for this change
More information about the conditions that caused a retry in the http
client. This information can help diagnose server or transport issues.
# What changes are included in this PR?
Updates to two `info!` logging statements and nothing else
# Are there any user-facing changes?
Yes, the logged messages will include new information. This shouldn't
require any new documentation.
<!--
If there are user-facing changes then we may require documentation to be
updated before approving the PR.
-->
<!---
If there are any breaking changes to public APIs, please call them out.
-->
--
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]
