refactor(storage): make indexes non-nullable#1543
Merged
Conversation
71225c6 to
4c3786c
Compare
2ec8c25 to
f81696f
Compare
bd714f8 to
e74ba1a
Compare
8c2ab1b to
ae67813
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1543 +/- ##
==========================================
+ Coverage 86.27% 86.29% +0.02%
==========================================
Files 437 437
Lines 33708 33605 -103
Branches 5260 5247 -13
==========================================
- Hits 29080 29001 -79
+ Misses 3611 3599 -12
+ Partials 1017 1005 -12 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
507778d to
d0984a1
Compare
e74ba1a to
2a11726
Compare
msbrogli
approved these changes
Jan 8, 2026
d0984a1 to
3d67ff1
Compare
2a11726 to
f30598f
Compare
3d67ff1 to
8636b4a
Compare
a2380e0 to
31e9bb0
Compare
1 task
31e9bb0 to
65d948d
Compare
ea0b002 to
0a281d2
Compare
84423ec to
b0f6fa0
Compare
5d82f32 to
4c46ed8
Compare
b0f6fa0 to
ba3cc55
Compare
468afc4 to
80c5302
Compare
The base branch was changed.
ba3cc55 to
88f1696
Compare
|
| Branch | refactor/require-indexes |
| Testbed | ubuntu-22.04 |
🚨 1 Alert
| Benchmark | Measure Units | View | Benchmark Result (Result Δ%) | Lower Boundary (Limit %) |
|---|---|---|---|---|
| sync-v2 (up to 20000 blocks) | Latency minutes (m) | 📈 plot 🚷 threshold 🚨 alert (🔔) | 1.36 m(-20.86%)Baseline: 1.72 m | 1.55 m (113.72%) |
Click to view all benchmark results
| Benchmark | Latency | Benchmark Result minutes (m) (Result Δ%) | Lower Boundary minutes (m) (Limit %) | Upper Boundary minutes (m) (Limit %) |
|---|---|---|---|---|
| sync-v2 (up to 20000 blocks) | 📈 view plot 🚷 view threshold 🚨 view alert (🔔) | 1.36 m(-20.86%)Baseline: 1.72 m | 1.55 m (113.72%) | 2.06 m (65.95%) |
jansegre
approved these changes
Jan 20, 2026
msbrogli
approved these changes
Jan 20, 2026
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Depends on #1542
Motivation
Previous PR #1542 removed the cache storage, which was the only reason
IndexesManagerwas nullable in the transaction storage. This PR addresses this, simplifying code, as it doesn't have to be nullable anymore. TheMempoolTipsIndexis also required, but code unnecessarily treated it as optional.Acceptance Criteria
IndexesManagernon-nullable in theTransactionStorage.indexes is not Noneandnot_none(indexes)assertions.MempoolTipsIndexnon-nullable in theIndexesManager.mempool_tips is not Noneandnot_none(mempool_tips)assertions.enable_mempool_index,force_clear_all,reset_indexes, and the_saving_genesisattribute.Checklist
master, confirm this code is production-ready and can be included in future releases as soon as it gets merged