HotSushi opened a new pull request, #2670: URL: https://github.com/apache/polaris/pull/2670
# Summary Previously, ServiceFailureException from persistence layer timeouts was being wrapped in AuthenticationFailedException, causing IOException during loadEntity operations to return 401 Unauthorized instead of the correct 503 Service Unavailable. This change allows ServiceFailureException to bubble up to IcebergExceptionMapper which properly maps it to 503 Service Unavailable, while preserving the existing behavior for other authentication exceptions. - Modified AuthenticatingAugmentor to catch ServiceFailureException separately - Added test case to verify ServiceFailureException bubbles up correctly - Other RuntimeExceptions still wrapped in AuthenticationFailedException for 401 # Tests ✅ Added a unit test ✅ `./gradlew build` succeeds -- 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]
