merlimat opened a new pull request, #25320: URL: https://github.com/apache/pulsar/pull/25320
## Summary - Fix `ConnectionTimeoutTest.testLowTimeout` which was flaky due to incorrect `CountDownLatch` count - The latch was initialized with `backlogSize + 1` but only `backlogSize` connections could succeed (the extra connection hangs because the server backlog is full), so the latch never reached zero - Changed the latch count to `backlogSize` so the test proceeds once all possible connections are established - Added debug logging to help diagnose future issues ## Test plan - [x] Run `ConnectionTimeoutTest.testLowTimeout` — should complete in seconds instead of timing out after 5 minutes ### Documentation - [ ] `doc-required` - [x] `doc-not-needed` - [ ] `doc-complete` ### Matching PR in forked repository PR in forked repository: <!-- ENTER URL HERE --> --- *This change is the fix for https://github.com/apache/pulsar/issues/15068 regression* -- 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]
