Skip to content

perf(migration): merge-join the headstate source instead of point reads - #4063

Open
NazariiDenha wants to merge 2 commits into
maksym/statehistory-migrationfrom
perf/state-headstate-iter
Open

NazariiDenha wants to merge 2 commits into
maksym/statehistory-migrationfrom
perf/state-headstate-iter

address comments

564f2ce
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 15, 2026 in 0s

79.19% (target 70.00%)

View this Pull Request on Codecov

79.19% (target 70.00%)

Details

Codecov Report

❌ Patch coverage is 80.58252% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.19%. Comparing base (ca4ebb2) to head (564f2ce).

Files with missing lines Patch % Lines
...ration/state/newstate/internal/headstate/source.go 80.26% 15 Missing ⚠️
...tion/state/newstate/internal/headstate/migrator.go 81.48% 5 Missing ⚠️
Additional details and impacted files
@@                        Coverage Diff                        @@
##           maksym/statehistory-migration    #4063      +/-   ##
=================================================================
- Coverage                          79.21%   79.19%   -0.02%     
=================================================================
  Files                                472      471       -1     
  Lines                              36089    36126      +37     
=================================================================
+ Hits                               28587    28610      +23     
- Misses                              7493     7507      +14     
  Partials                               9        9              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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