xline-kv / Xline

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

feat: implement WAL segments remover #680

Closed bsbds closed 1 month ago

bsbds commented 4 months ago

Depends-on: #705

Please briefly answer these questions:

bsbds commented 3 months ago

Rebased on #705

mergify[bot] commented 2 months ago

@bsbds Convert your pr to draft since CI failed

mergify[bot] commented 2 months ago

@bsbds Convert your pr to draft since CI failed

mergify[bot] commented 2 months ago

@bsbds Convert your pr to draft since CI failed

Phoenix500526 commented 2 months ago

@Mergifyio rebase

mergify[bot] commented 2 months ago

rebase

✅ Branch has been successfully rebased

mergify[bot] commented 2 months ago

@bsbds Convert your pr to draft since CI failed

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 86.86131% with 18 lines in your changes are missing coverage. Please review.

Project coverage is 75.64%. Comparing base (e35b35a) to head (26fdc28). Report is 97 commits behind head on master.

Files Patch % Lines
crates/curp/src/server/storage/wal/remover.rs 86.86% 7 Missing and 11 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #680 +/- ## ========================================== + Coverage 75.55% 75.64% +0.09% ========================================== Files 180 187 +7 Lines 26938 27625 +687 Branches 26938 27625 +687 ========================================== + Hits 20353 20897 +544 - Misses 5366 5445 +79 - Partials 1219 1283 +64 ```

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

mergify[bot] commented 2 months ago

@bsbds Convert your pr to draft since CI failed

mergify[bot] commented 1 month ago

@bsbds Your PR is in conflict and cannot be merged.