xiangfu0 opened a new pull request, #18058: URL: https://github.com/apache/pinot/pull/18058
## Summary - Fix QueryRoutingTest which had **12 failures** (4x each for `testServerDown`, `testValidResponse`, `testSkipUnavailableServer`) in the weekly CI digest (2026-03-22 to 2026-03-28) - Replace hardcoded ports (12345/12346/12347) with dynamic `ServerSocket(0)` allocation - Add `startQueryServerWithWait()` for proper server readiness synchronization before sending queries - Improve `@AfterMethod` shutdown with exception handling and port release delay ## Test plan - [ ] Run `QueryRoutingTest` locally multiple times to verify no port conflict failures - [ ] Verify all 9 test methods pass consistently - [ ] Monitor next week's flaky test digest for regression 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
