yadavay-amzn commented on PR #16049: URL: https://github.com/apache/iceberg/pull/16049#issuecomment-4306269328
Updated. The new tests directly demonstrate the retry behavior difference: - `testSQLExceptionEnablesRetryInClientPool`: lambda throws `SQLTransientConnectionException` on first call, succeeds on retry. Passes only when `SQLException` propagates naturally (fix applied). - `testUncheckedSQLExceptionBypassesRetry`: same exception wrapped as `UncheckedSQLException` (old behavior) — `ClientPoolImpl` cannot catch `RuntimeException`, so no retry happens. -- 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]
