MrCroxx / RunKV

[WIP] An experimental cloud-native distributed KV engine for OLTP workload.
MIT License
68 stars 7 forks source link

fix: fix raft log store read race #167

Closed MrCroxx closed 2 years ago

MrCroxx commented 2 years ago

What changes were proposed in this pull request?

Changes:

Which issues is this PR related to?

Fix: #168 .

codecov[bot] commented 2 years ago

Codecov Report

Merging #167 (3085344) into main (1d3c69f) will decrease coverage by 0.03%. The diff coverage is 73.10%.

@@            Coverage Diff             @@
##             main     #167      +/-   ##
==========================================
- Coverage   83.88%   83.84%   -0.04%     
==========================================
  Files          85       86       +1     
  Lines        8382     8637     +255     
==========================================
+ Hits         7031     7242     +211     
- Misses       1351     1395      +44     
Impacted Files Coverage Δ
wheel/src/components/fsm.rs 70.74% <0.00%> (-0.32%) :arrow_down:
storage/src/raft_log_store/log.rs 81.85% <63.33%> (-5.60%) :arrow_down:
storage/src/raft_log_store/mem.rs 80.96% <65.38%> (-0.76%) :arrow_down:
storage/src/raft_log_store/file.rs 76.31% <76.31%> (ø)
storage/src/raft_log_store/store.rs 92.70% <86.11%> (+2.88%) :arrow_up:
storage/src/raft_log_store/entry.rs 89.38% <100.00%> (+0.03%) :arrow_up:
wheel/src/lib.rs 90.59% <0.00%> (-0.43%) :arrow_down:
wheel/src/worker/raft.rs 94.62% <0.00%> (ø)
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1d3c69f...3085344. Read the comment docs.