tikv / raft-engine

A persistent storage engine for Multi-Raft log
Apache License 2.0
565 stars 88 forks source link

fix bugs related to purge rewrite #160

Closed tabokie closed 2 years ago

tabokie commented 2 years ago

~It's possible that certain raft groups are not compacted due to upper level issues. Those laggers will block the garbage collection progress indefinitely. After this patch, they will be rewritten regardless of the amount of their containing entries.~

Fix three bugs found during stability tests:

codecov[bot] commented 2 years ago

Codecov Report

Merging #160 (0e1aed3) into master (a06397b) will decrease coverage by 0.06%. The diff coverage is 94.31%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #160      +/-   ##
==========================================
- Coverage   97.01%   96.94%   -0.07%     
==========================================
  Files          26       26              
  Lines        6523     6546      +23     
==========================================
+ Hits         6328     6346      +18     
- Misses        195      200       +5     
Impacted Files Coverage Δ
src/engine.rs 92.97% <ø> (-0.05%) :arrow_down:
src/purge.rs 96.09% <85.29%> (-1.40%) :arrow_down:
src/memtable.rs 99.64% <100.00%> (+<0.01%) :arrow_up:
src/metrics.rs 97.47% <100.00%> (+0.11%) :arrow_up:
tests/failpoints/test_engine.rs 100.00% <100.00%> (ø)
src/codec.rs 98.34% <0.00%> (ø)

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 a06397b...0e1aed3. Read the comment docs.