williamhyun commented on PR #17457: URL: https://github.com/apache/iceberg/pull/17457#issuecomment-5273731523
Thank you @xndai for the comments, I have applied the feedback and would appreciate if you could take another look! Changes made: - Centralized all HTTP status handling in a single HttpStatusCategory.classify() - Tightened classification for transient and terminal errors. - Allow-listed S3 host check before reads - Replaced the manual retry loop with Tasks-based exponential backoff -- 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]
