Skip to content

Removes AccountsDb::accounts_delta_hash#7095

Merged
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:lthash/accounts-delta-hash
Jul 23, 2025
Merged

Removes AccountsDb::accounts_delta_hash#7095
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:lthash/accounts-delta-hash

Conversation

@brooksprumo
Copy link
Copy Markdown

@brooksprumo brooksprumo commented Jul 23, 2025

Problem

The merkle-based accounts hashing is no longer used.

Summary of Changes

This PR removes the AccountsDb::accounts_delta_hash field.

@brooksprumo brooksprumo self-assigned this Jul 23, 2025
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.2%. Comparing base (e343d15) to head (f67108e).

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #7095     +/-   ##
=========================================
- Coverage    83.2%    83.2%   -0.1%     
=========================================
  Files         853      853             
  Lines      374131   374084     -47     
=========================================
- Hits       311510   311467     -43     
+ Misses      62621    62617      -4     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@brooksprumo brooksprumo force-pushed the lthash/accounts-delta-hash branch from 14b4b3d to ef53649 Compare July 23, 2025 17:22
@brooksprumo brooksprumo force-pushed the lthash/accounts-delta-hash branch from ef53649 to f67108e Compare July 23, 2025 18:11
@brooksprumo brooksprumo marked this pull request as ready for review July 23, 2025 18:11
@brooksprumo brooksprumo requested review from HaoranYi and roryharr July 23, 2025 18:11
Copy link
Copy Markdown

@HaoranYi HaoranYi left a comment

Choose a reason for hiding this comment

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

lgtm.

@brooksprumo brooksprumo merged commit 1573bd3 into anza-xyz:master Jul 23, 2025
41 checks passed
@brooksprumo brooksprumo deleted the lthash/accounts-delta-hash branch July 23, 2025 20:18
puhtaytow pushed a commit to puhtaytow/agave that referenced this pull request Jul 24, 2025
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