-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix zookeeper registry get wrong provider list. (#166) #174
Conversation
7abddca
to
13bbe7c
Compare
e0e5484
to
58ed594
Compare
Codecov Report
@@ Coverage Diff @@
## master #174 +/- ##
============================================
+ Coverage 70.99% 71.26% +0.26%
Complexity 955 955
============================================
Files 341 341
Lines 14196 14210 +14
Branches 2281 2286 +5
============================================
+ Hits 10079 10127 +48
+ Misses 2939 2891 -48
- Partials 1178 1192 +14
Continue to review full report at Codecov.
|
* @param <T> 值类型 | ||
* @return 返回值 | ||
*/ | ||
protected <T> T delayGet(Callable<T> callable, T expect, int period, int times) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to add this method because I add TestUtils.java
at #180 contains this method. Just use TestUtils.delayGet()
instead of it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, I have fixed it.
Result:
Fixes #166