-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
enhance: avoid the coexistence of the old coordinator and the new node/proxy #35720
enhance: avoid the coexistence of the old coordinator and the new node/proxy #35720
Conversation
7bd2f31
to
5dc32bc
Compare
/run-cpu-e2e |
@weiliu1031 E2e jenkins job failed, comment |
5dc32bc
to
36edb6e
Compare
@weiliu1031 E2e jenkins job failed, comment |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #35720 +/- ##
==========================================
+ Coverage 81.32% 84.66% +3.34%
==========================================
Files 1238 960 -278
Lines 148082 123447 -24635
==========================================
- Hits 120425 104516 -15909
+ Misses 22887 14154 -8733
- Partials 4770 4777 +7 |
…e/proxy In standalone mode, block the start process until the new coordinator is active to avoid the coexistence of the old coordinator and the new node/proxy 1. In the start/restart process, the new coordinator will become active immediately and will not be blocked 2. In the rolling upgrade process, the new coordinator will not be active until the old coordinator is down, and it will be blocked Signed-off-by: Wei Liu <wei.liu@zilliz.com>
36edb6e
to
4367a7e
Compare
@weiliu1031 E2e jenkins job failed, comment |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: weiliu1031, xiaofan-luan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…e/proxy (#35760) issue: #35719 pr: #35720 In standalone mode, block the start process until the new coordinator is active to avoid the coexistence of the old coordinator and the new node/proxy 1. In the start/restart process, the new coordinator will become active immediately and will not be blocked 2. In the rolling upgrade process, the new coordinator will not be active until the old coordinator is down, and it will be blocked Signed-off-by: Wei Liu <wei.liu@zilliz.com>
/run-cpu-e2e |
@weiliu1031 E2e jenkins job failed, comment |
/run-cpu-e2e |
…d the new node/proxy (milvus-io#35720)" This reverts commit 73be0ba.
…d the new node/proxy (milvus-io#35720)" This reverts commit 73be0ba.
…d the new node/proxy (milvus-io#35720)" This reverts commit 73be0ba. Signed-off-by: Wei Liu <wei.liu@zilliz.com>
…d the new node/proxy (milvus-io#35720)" (milvus-io#36113) issue: milvus-io#36100 This reverts commit 73be0ba. Signed-off-by: Wei Liu <wei.liu@zilliz.com>
issue: #35719
In standalone mode, block the start process until the new coordinator is active to avoid the coexistence of the old coordinator and the new node/proxy
1. In the start/restart process, the new coordinator will become active immediately and will not be blocked
2. In the rolling upgrade process, the new coordinator will not be active until the old coordinator is down, and it will be blocked