Skip to content

fix(metadata): truncate the WAL suffix above the state-transfer floor - #3918

Open
numinnex wants to merge 7 commits into
masterfrom
fix_metadata_wal_truncate
Open

fix(metadata): truncate the WAL suffix above the state-transfer floor#3918
numinnex wants to merge 7 commits into
masterfrom
fix_metadata_wal_truncate

Conversation

@numinnex

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the S-waiting-on-review PR is waiting on a reviewer label Aug 18, 2026
@numinnex

Copy link
Copy Markdown
Contributor Author

Merge after #3915

@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 79.65057% with 198 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.35%. Comparing base (3cd0860) to head (120acfb).

Files with missing lines Patch % Lines
core/server/src/partition_reconciler.rs 45.07% 106 Missing ⚠️
core/server/src/dispatch.rs 82.70% 18 Missing and 5 partials ⚠️
core/server/src/bootstrap.rs 80.64% 11 Missing and 1 partial ⚠️
core/partitions/src/iggy_index_reader.rs 77.08% 9 Missing and 2 partials ⚠️
core/simulator/src/lib.rs 76.59% 11 Missing ⚠️
core/server/src/http/metrics.rs 50.00% 10 Missing ⚠️
core/shard/src/router.rs 70.00% 5 Missing and 1 partial ⚠️
core/configs/src/server_config/sharding.rs 58.33% 5 Missing ⚠️
core/server_common/src/executor.rs 86.11% 5 Missing ⚠️
core/configs/src/server_config/cluster.rs 90.90% 1 Missing ⚠️
... and 8 more
Additional details and impacted files
@@              Coverage Diff              @@
##             master    #3918       +/-   ##
=============================================
- Coverage     83.76%   64.35%   -19.42%     
  Complexity     1358     1358               
=============================================
  Files          1212     1209        -3     
  Lines        166189   153983    -12206     
  Branches     133663   121790    -11873     
=============================================
- Hits         139210    99094    -40116     
- Misses        23342    51170    +27828     
- Partials       3637     3719       +82     
Components Coverage Δ
Rust Core 61.04% <79.65%> (-23.51%) ⬇️
Java SDK 66.67% <ø> (ø)
C# SDK 65.88% <ø> (-10.21%) ⬇️
Python SDK 90.00% <ø> (ø)
PHP SDK 84.48% <ø> (ø)
Node SDK 89.09% <ø> (-6.76%) ⬇️
Go SDK 68.32% <ø> (ø)
Files with missing lines Coverage Δ
core/configs/src/common/server.rs 100.00% <100.00%> (ø)
core/configs/src/server_config/defaults.rs 100.00% <100.00%> (ø)
core/consensus/src/impls.rs 82.33% <ø> (-8.84%) ⬇️
core/journal/src/lib.rs 25.00% <ø> (ø)
core/journal/src/prepare_journal.rs 70.58% <ø> (-19.87%) ⬇️
core/partitions/src/iggy_partition.rs 70.13% <100.00%> (-17.18%) ⬇️
core/partitions/src/journal.rs 79.40% <ø> (-9.99%) ⬇️
core/partitions/src/lib.rs 0.00% <ø> (ø)
core/partitions/src/log.rs 71.67% <100.00%> (-9.25%) ⬇️
core/server/src/auth.rs 80.65% <100.00%> (-9.47%) ⬇️
... and 31 more

... and 464 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review PR is waiting on a reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant