-
Notifications
You must be signed in to change notification settings - Fork 76
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
feature: add apply_conf_change
method for RawCurp
#433
Conversation
af048e0
to
b72ffd3
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #433 +/- ##
==========================================
+ Coverage 54.02% 54.29% +0.27%
==========================================
Files 102 102
Lines 17199 17378 +179
Branches 17199 17378 +179
==========================================
+ Hits 9291 9436 +145
- Misses 7341 7369 +28
- Partials 567 573 +6
☔ View full report in Codecov by Sentry. |
b72ffd3
to
b5087f5
Compare
b5087f5
to
da247f8
Compare
e3ab157
to
d28e825
Compare
BTW, can we directly use the etcd's protobuf definition of ConfChange? This way, we won't need to add an extra layer of translation in xline to the current definition of ConfChange. |
00d5395
to
2780347
Compare
refactor data structures that need to be mutable Signed-off-by: themanforfree <themanforfree@gmail.com>
Signed-off-by: themanforfree <themanforfree@gmail.com>
Signed-off-by: themanforfree <themanforfree@gmail.com>
Signed-off-by: themanforfree <themanforfree@gmail.com>
2780347
to
4437b4b
Compare
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.
This PR should be based on PR #451 and PR xline-kv/curp-proto#2
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.
LGTM
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.
LGTM
Please briefly answer these questions:
#306
apply config change to curp
refactor data structures via
DashMap
, and modify them when apply_conf_changeno