xiangfu0 opened a new pull request, #19133: URL: https://github.com/apache/pinot/pull/19133
## Summary - Scope the 10-second segment-completion deadline to the controller failure-injection tests that require it. - Configure the short deadline only after the non-pauseless comparison table finishes setup. - Keep the short deadline through the explicit validation repair pass, then restore the normal 5-minute deadline. ## Root cause `BasePauselessRealtimeIngestionTest` globally replaced the controller segment-completion deadline with 10 seconds. That affected ordinary pauseless tests, table rebalance tests, and the non-pauseless comparison table even when no controller fault was injected. On a busy CI runner, normal segment commits exceeded the artificial deadline and were repeatedly repaired, causing otherwise unrelated tests to hit their 20-minute timeout. ## How to reproduce Run the Set 2 integration-test shard containing the pauseless suites under CI load. In [the failing job](https://github.com/apache/pinot/actions/runs/30516372127/job/90787057324?pr=19091), the basic pauseless test logged 38 segment-completion deadline expirations, the rebalance test logged 14, and the ideal-state failure test logged 2 before five tests timed out. The [passing run on the same branch](https://github.com/apache/pinot/actions/runs/30509418502/job/90766056417) and a [passing master run](https://github.com/apache/pinot/actions/runs/30503923617/job/90749450030) logged no such expirations for those classes. ## Testing - JDK 25 focused reactor run: `./mvnw -pl pinot-integration-tests -am -Dtest=PauselessRealtimeIngestionIntegrationTest,TableRebalancePauselessIntegrationTest,PauselessRealtimeIngestionIdealStateUpdateFailureTest,PauselessRealtimeIngestionNewSegmentMetadataCreationFailureTest,PauselessRealtimeIngestionCommitEndMetadataFailureTest -Dsurefire.failIfNoSpecifiedTests=false test` (6 tests, 0 failures) - `./mvnw spotless:apply -pl pinot-integration-tests` - `./mvnw license:format -pl pinot-integration-tests` - `./mvnw checkstyle:check -pl pinot-integration-tests` - `./mvnw license:check -pl pinot-integration-tests` - JDK 25 warning-enabled reactor `test-compile` with `-Xlint:all` -- 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]
