Skip to content

indexer: byte-aware admission - #45

Closed
michaelkaplan13 wants to merge 1 commit into
michael/qmdb-suffix-recoveryfrom
michael/indexer-byte-admission
Closed

michaelkaplan13 wants to merge 1 commit into
michael/qmdb-suffix-recoveryfrom
michael/indexer-byte-admission

Conversation

@michaelkaplan13

Copy link
Copy Markdown
Contributor

No description provided.

@michaelkaplan13 michaelkaplan13 changed the title Michael/indexer byte admission Indexer byte-aware admission Aug 13, 2026
@michaelkaplan13 michaelkaplan13 changed the title Indexer byte-aware admission indexer: byte-aware admission Aug 13, 2026
@michaelkaplan13 michaelkaplan13 self-assigned this Aug 14, 2026
@michaelkaplan13
michaelkaplan13 force-pushed the michael/indexer-byte-admission branch from a5c8da9 to efc0eb2 Compare August 24, 2026 20:48
- wait for finalized upload persistence
- defer finalized upload decoding
- bound finalized upload memory by bytes
- fix finalized upload enqueue ordering
@michaelkaplan13
michaelkaplan13 force-pushed the michael/indexer-byte-admission branch from efc0eb2 to 5641fd9 Compare August 24, 2026 21:21
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.27166% with 66 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.10%. Comparing base (230b1a8) to head (5641fd9).

Files with missing lines Patch % Lines
bin/validator/src/run.rs 90.01% 63 Missing ⚠️
crates/indexer/src/publisher/qmdb.rs 96.90% 3 Missing ⚠️
@@                       Coverage Diff                        @@
##           michael/qmdb-suffix-recovery      #45      +/-   ##
================================================================
+ Coverage                         78.07%   80.10%   +2.02%     
================================================================
  Files                                52       52              
  Lines                             13006    13783     +777     
================================================================
+ Hits                              10155    11041     +886     
+ Misses                             2851     2742     -109     
Files with missing lines Coverage Δ
bin/deploy/src/local.rs 90.57% <100.00%> (+0.03%) ⬆️
bin/deploy/src/main.rs 82.71% <ø> (ø)
bin/deploy/src/remote.rs 79.39% <100.00%> (+0.05%) ⬆️
bin/validator/src/config.rs 97.39% <100.00%> (+0.04%) ⬆️
crates/indexer/src/publisher/certificate.rs 68.62% <100.00%> (+52.35%) ⬆️
crates/indexer/src/publisher/qmdb.rs 96.56% <96.90%> (+1.28%) ⬆️
bin/validator/src/run.rs 65.64% <90.01%> (+24.93%) ⬆️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 230b1a8...5641fd9. Read the comment docs.

🚀 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.

@michaelkaplan13

Copy link
Copy Markdown
Contributor Author

Going to be replaced with a new stack with fewer intermediate states. Closing

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant