DanielLeens opened a new pull request, #11425: URL: https://github.com/apache/seatunnel/pull/11425
## Purpose The coordinator retry test depended on Hazelcast's production default retry budget and pause. Exercising the full default budget makes an otherwise deterministic unit test unnecessarily slow and vulnerable to CI timing variation. ## Changes - Configure a test-only maximum of three invocation attempts with a one-millisecond retry pause. - Reset and expose the test operation attempt counter for an exact terminal assertion. - Preserve the original contract by waiting for the terminal `CompletionException`, checking its retry message, and asserting the exact attempt count. - Always shut down the test Hazelcast instance in `finally`. ## Validation - `CoordinatorServiceTest#testSeaTunnelEngineRetryableExceptionOperationCanBeRetryByHazelcast`: 1 test, 0 failures, 0 errors - `verify -DskipTests` for `seatunnel-engine-server` (includes test compilation and Spotless check) - Two independent code reviews passed with no blocking findings `seatunnel-engine-ui` was excluded because this change does not touch the UI. -- 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]
