Skip to content

Removes calculate_accounts_delta_hash()#7093

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

Removes calculate_accounts_delta_hash()#7093
brooksprumo merged 1 commit intoanza-xyz:masterfrom
brooksprumo:lthash/calc-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 calculate_accounts_delta_hash().

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 23, 2025

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 83.2%. Comparing base (6a4b1fb) to head (d62324b).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #7093     +/-   ##
=========================================
- Coverage    83.2%    83.2%   -0.1%     
=========================================
  Files         854      854             
  Lines      374326   374207    -119     
=========================================
- Hits       311763   311595    -168     
- Misses      62563    62612     +49     
🚀 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/calc-delta-hash branch from d0c5270 to d62324b Compare July 23, 2025 16:01
@brooksprumo brooksprumo marked this pull request as ready for review July 23, 2025 16:05
@brooksprumo brooksprumo requested review from HaoranYi and roryharr July 23, 2025 16:05
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 e343d15 into anza-xyz:master Jul 23, 2025
41 checks passed
@brooksprumo brooksprumo deleted the lthash/calc-delta-hash branch July 23, 2025 18:10
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