Skip to content

Split update_index function into two#9963

Merged
roryharr merged 3 commits intoanza-xyz:masterfrom
roryharr:refactor_update_index_cached
Jan 14, 2026
Merged

Split update_index function into two#9963
roryharr merged 3 commits intoanza-xyz:masterfrom
roryharr:refactor_update_index_cached

Conversation

@roryharr
Copy link
Copy Markdown

@roryharr roryharr commented Jan 13, 2026

Problem

Update Index is used for both cached accounts and uncached accounts which leads to unnecessary data structure allocations (reclaims). This is also a building block towards ephemeral accounts skipping index updates

Summary of Changes

  • Created a new function refactor_update_index_cached

Fixes #

@roryharr roryharr force-pushed the refactor_update_index_cached branch 2 times, most recently from 825c546 to 4cc5cf7 Compare January 13, 2026 02:07
@roryharr roryharr force-pushed the refactor_update_index_cached branch from 4cc5cf7 to 5e201d9 Compare January 13, 2026 17:23
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.5%. Comparing base (552674f) to head (259ac21).
⚠️ Report is 11 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #9963     +/-   ##
=========================================
- Coverage    82.5%    82.5%   -0.1%     
=========================================
  Files         844      844             
  Lines      316472   316510     +38     
=========================================
+ Hits       261309   261335     +26     
- Misses      55163    55175     +12     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@roryharr roryharr marked this pull request as ready for review January 14, 2026 00:21
@roryharr roryharr requested a review from brooksprumo January 14, 2026 00:22
Copy link
Copy Markdown

@brooksprumo brooksprumo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

Comment thread accounts-db/src/accounts_db.rs
Comment thread accounts-db/src/accounts_db.rs Outdated
@roryharr roryharr added this pull request to the merge queue Jan 14, 2026
Merged via the queue into anza-xyz:master with commit 87ed021 Jan 14, 2026
47 checks passed
@roryharr roryharr deleted the refactor_update_index_cached branch January 14, 2026 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants