-
Notifications
You must be signed in to change notification settings - Fork 930
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: fix the black list bug and make ConnCheckRouter work well #1287
Conversation
1. change pageSize to 64; 2. make the black recover by Service: different service can recover at the same time. 3. fix routeChain.notify just global lock. The lock should also in service Dimension
make the blackList belong to one service.
Codecov Report
@@ Coverage Diff @@
## 1.5 #1287 +/- ##
==========================================
- Coverage 66.13% 58.07% -8.07%
==========================================
Files 174 275 +101
Lines 9343 13381 +4038
==========================================
+ Hits 6179 7771 +1592
- Misses 2541 4650 +2109
- Partials 623 960 +337
Continue to review full report at Codecov.
|
The feature about router is not supported in 3.0, so I will not submit to 3.0.