pcmoritz opened a new pull request, #14019:
URL: https://github.com/apache/arrow/pull/14019

   This is the last change I propose to improve our S3 error message.
   
   For certain errors, unfortunately the AWS SDK is doing a poor job in 
propagating the error and just reports UNKNOWN (see 
https://github.com/aws/aws-sdk-cpp/blob/1614bce979a201ada1e3436358edb7bd1834b5d6/aws-cpp-sdk-core/source/client/AWSClient.cpp#L77),
 in these cases the HTTP status code can be an important source to find out 
what is going wrong (and is also reported by boto3).
   
   This has the downside of cluttering the error message a bit more, but in 
general this information will be very valuable to diagnose the problem. Given 
that we now have the API call and the HTTP status error, in general there is 
good documentation on the internet that helps diagnose the problem.
   
   Before:
   
   > When getting information for key 'test.csv' in bucket 
'pcmoritz-test-bucket-arrow-errors': AWS Error UNKNOWN during HeadObject call: 
No response body.
   
   After:
   
   > When getting information for key 'test.csv' in bucket 
'pcmoritz-test-bucket-arrow-errors': AWS Error UNKNOWN **(http status code: 
400)** during HeadObject call: No response body.


-- 
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]

Reply via email to