Skip to content

Clean up hold_range_in_memory from accounts index: part 2#6979

Merged
HaoranYi merged 1 commit intoanza-xyz:masterfrom
HaoranYi:range_p2
Jul 16, 2025
Merged

Clean up hold_range_in_memory from accounts index: part 2#6979
HaoranYi merged 1 commit intoanza-xyz:masterfrom
HaoranYi:range_p2

Conversation

@HaoranYi
Copy link
Copy Markdown

@HaoranYi HaoranYi commented Jul 15, 2025

Problem

hold_range_in_memory was originally introduced to support rent scanning by keeping certain key ranges in memory. However, since we no longer perform range-based rent scanning, this mechanism is now obsolete.
This PR is part 2 for the clean up: remove hold_range check from evict.

Summary of Changes

  • remove hold_range check from evict

Fixes #

@HaoranYi HaoranYi changed the title remove range check from evict Clean up hold_range_in_memory from accounts index: part 2 Jul 15, 2025
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.2%. Comparing base (d429f6e) to head (3887c51).
Report is 7 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #6979    +/-   ##
========================================
  Coverage    83.2%    83.2%            
========================================
  Files         856      853     -3     
  Lines      376421   375577   -844     
========================================
- Hits       313240   312649   -591     
+ Misses      63181    62928   -253     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

:shipit:

@HaoranYi HaoranYi merged commit 5604510 into anza-xyz:master Jul 16, 2025
41 checks passed
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.

4 participants