xline-kv / Xline

A geo-distributed KV store for metadata management
https://xline.cloud
Apache License 2.0
596 stars 74 forks source link

fix: fix the calculation logic of superquorum #636

Closed Phoenix500526 closed 8 months ago

Phoenix500526 commented 8 months ago

Please briefly answer these questions: Closes #635

codecov[bot] commented 8 months ago

Codecov Report

Attention: 625 lines in your changes are missing coverage. Please review.

Comparison is base (fa656ad) 64.40% compared to head (ec646a2) 79.79%. Report is 112 commits behind head on master.

Files Patch % Lines
curp/src/client/unary.rs 85.92% 48 Missing :warning:
curp/src/server/curp_node.rs 82.75% 45 Missing :warning:
curp/src/server/raw_curp/mod.rs 83.92% 45 Missing :warning:
xline/src/storage/alarm_store.rs 67.88% 44 Missing :warning:
curp/src/rpc/connect.rs 70.00% 33 Missing :warning:
utils/src/parser.rs 84.79% 33 Missing :warning:
xline/src/storage/kv_store.rs 23.07% 30 Missing :warning:
xline/src/server/maintenance.rs 63.29% 29 Missing :warning:
curp/src/client/state.rs 88.66% 23 Missing :warning:
curp/src/client/tests.rs 94.73% 22 Missing :warning:
... and 30 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #636 +/- ## =========================================== + Coverage 64.40% 79.79% +15.39% =========================================== Files 105 154 +49 Lines 18802 22914 +4112 Branches 18802 0 -18802 =========================================== + Hits 12109 18285 +6176 + Misses 5955 4629 -1326 + Partials 738 0 -738 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.