SURYAS1306 opened a new pull request, #16086:
URL: https://github.com/apache/dubbo/pull/16086
Fixes #16085
## What is the purpose of the change?
The Windows Zookeeper startup logic currently relies on a fixed sleep:
TimeUnit.SECONDS.sleep(3);
This is unreliable because Zookeeper startup time varies depending on machine
performance and environment. As a result, tests may randomly fail when the
server is not fully ready after the fixed delay.
This PR replaces the fixed delay with an active readiness check using
CuratorFramework.blockUntilConnected(). The processor now waits until the
Zookeeper instance is actually reachable before continuing.
This makes the test startup deterministic and removes flaky behavior.
## Brief changelog
- Remove fixed 3-second sleep
- Add Curator-based readiness check
- Wait for each client port to become available
- Improve Windows test stability
## Verifying this change
- Project builds successfully
- Module `dubbo-test-check` compiles
- No behavior change except improved stability
- Prevents intermittent startup failures
## Checklist
- [x] Make sure there is a GitHub issue field for the change.
- [x] Write a pull request description that is detailed enough to understand
what the pull request does, how, and why.
- [x] No new feature introduced
- [x] Code style and format verified (spotless applied)
--
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]
