Visitar URL original
fix(shard): keep the pipeline head when the metadata walk stalls by numinnex · Pull Request #4429 · apache/iggy · GitHub
Skip to content

fix(shard): keep the pipeline head when the metadata walk stalls - #4429

Open
numinnex wants to merge 2 commits into
masterfrom
fix_journal_panic
Open

numinnex wants to merge 2 commits into
masterfrom
fix_journal_panic

Conversation

@numinnex

@numinnex numinnex commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

Closes #4294

@github-actions github-actions Bot added the S-waiting-on-review PR is waiting on a reviewer label Oct 7, 2026
@codecov

codecov Bot commented Oct 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 99.31507% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 77.79%. Comparing base (07d2a21) to head (d38a3dc).
⚠️ Report is 5 commits behind head on master.

Files with missing lines Patch % Lines
core/simulator/src/lib.rs 98.07% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #4429      +/-   ##
============================================
- Coverage     87.53%   77.79%   -9.75%     
  Complexity     1606     1606              
============================================
  Files          1299     1295       -4     
  Lines        241165   176833   -64332     
  Branches     201569   137239   -64330     
============================================
- Hits         211109   137569   -73540     
- Misses        25014    34329    +9315     
+ Partials       5042     4935     -107     
Components Coverage Δ
Rust Core 76.21% <99.31%> (-13.07%) ⬇️
Java SDK 68.72% <ø> (ø)
C# SDK 77.83% <ø> (+16.39%) ⬆️
Python SDK 91.25% <ø> (ø)
PHP SDK 85.67% <ø> (ø)
Node SDK 96.51% <ø> (-0.11%) ⬇️
Go SDK 70.30% <ø> (+0.05%) ⬆️
C++ SDK 86.60% <ø> (ø)
Files with missing lines Coverage Δ
core/metadata/src/impls/metadata.rs 91.46% <100.00%> (+0.74%) ⬆️
core/shard/src/lib.rs 86.78% <100.00%> (-0.01%) ⬇️
core/simulator/src/lib.rs 97.04% <98.07%> (+0.01%) ⬆️

... and 441 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.

This branch has not been deployed

No deployments
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.

Metadata primary panics "committed prepare ... must be in journal" after its walk-stall detector truncates the pipeline head

1 participant