search

tikv / raft-engine

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

Fix two cases of false positive corruption #191

Closed tabokie closed 2 years ago

tabokie commented 2 years ago

Two cases:

Patched the mocking filesystem to cover these cases.

codecov[bot] commented 2 years ago

Codecov Report

Merging #191 (97aca0f) into master (11ab879) will increase coverage by 0.13%. The diff coverage is 96.25%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #191      +/-   ##
==========================================
+ Coverage   96.77%   96.90%   +0.13%     
==========================================
  Files          28       28              
  Lines        7346     7408      +62     
==========================================
+ Hits         7109     7179      +70     
+ Misses        237      229       -8
Impacted Files Coverage Δ
src/file_pipe_log/pipe_builder.rs 96.92% <70.00%> (+2.29%) :arrow_up:
src/engine.rs 97.07% <100.00%> (+0.12%) :arrow_up:
src/env/obfuscated.rs 93.75% <100.00%> (+0.13%) :arrow_up:
src/file_pipe_log/log_file.rs 100.00% <100.00%> (ø)
src/consistency.rs 91.11% <0.00%> (-2.23%) :arrow_down:
src/env/default.rs 93.78% <0.00%> (+0.56%) :arrow_up:
src/file_pipe_log/reader.rs 96.26% <0.00%> (+3.73%) :arrow_up:

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 11ab879...97aca0f. Read the comment docs.