Skip to content

Removes merkle-based accounts hashing in AccountsHashVerifier#7035

Merged
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:lthash/220/ahv/merkle
Jul 21, 2025
Merged

Removes merkle-based accounts hashing in AccountsHashVerifier#7035
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:lthash/220/ahv/merkle

Conversation

@brooksprumo
Copy link
Copy Markdown

@brooksprumo brooksprumo commented Jul 18, 2025

Problem

The merkle-based accounts hashing is no longer used.

Summary of Changes

For this PR, we remove the merkle-based accounts hashing from AccountsHashVerifier.

@brooksprumo brooksprumo self-assigned this Jul 18, 2025
@brooksprumo brooksprumo force-pushed the lthash/220/ahv/merkle branch from 90aa117 to ee85478 Compare July 18, 2025 16:52
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 18, 2025

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 83.2%. Comparing base (f5c84e3) to head (0216a57).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #7035    +/-   ##
========================================
  Coverage    83.2%    83.2%            
========================================
  Files         853      853            
  Lines      374698   374523   -175     
========================================
- Hits       311796   311769    -27     
+ Misses      62902    62754   -148     
🚀 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/220/ahv/merkle branch from ee85478 to 0216a57 Compare July 18, 2025 18:00
@brooksprumo brooksprumo marked this pull request as ready for review July 18, 2025 18:21
@brooksprumo brooksprumo requested review from HaoranYi and roryharr July 18, 2025 18:28
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 3122ab7 into anza-xyz:master Jul 21, 2025
41 checks passed
@brooksprumo brooksprumo deleted the lthash/220/ahv/merkle branch July 21, 2025 16:52
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