steveloughran commented on PR #6938:
URL: https://github.com/apache/hadoop/pull/6938#issuecomment-2327346018

   today's changes. 
   
   * fix failing test ITestS3AConcurrentOps
   * General review of S3ABlockOutputStream, inc javadocs
     and Retries attributes
   * S3ADataBlocks also has its methods' exceptions reviewed and
     javadocs updated
   * Default value of fs.s3a.retry.http.5xx.errors is true
   * Troubleshooting and third-party docs updated
   
   The javadocs are mainly @throws calls as I went down them all to see what 
could be thrown and why, just to reassure myself there's no other causes of 
cancellation.
   
   The change of `fs.s3a.retry.http.5xx.errors` default to true is probably the 
most controversial as mukund felt the SDK retried anyway and it is so rare.
   I think we should do it just in case -and I've updated the third party and 
troubleshooting
   docs to match and discuss turning it off.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to