search

anza-xyz / agave

Web-Scale Blockchain for fast, secure, scalable, decentralized apps and marketplaces.
https://www.anza.xyz/
Apache License 2.0
178 stars 67 forks source link

refactor appendvec.get_account_shared_data #1419

Closed jeffwashington closed 1 week ago

jeffwashington commented 2 weeks ago

Problem

moving to using file i/o for append vecs

Summary of Changes

refactor appendvec.get_account_shared_data to re-use get_stored_account_meta_callback. There won't be any measurable perf penalty but implementation and testing are improved.

Fixes #

codecov-commenter commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.7%. Comparing base (83be26a) to head (326270b). Report is 4 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1419 +/- ## ========================================= - Coverage 82.7% 82.7% -0.1% ========================================= Files 871 871 Lines 370276 370243 -33 ========================================= - Hits 306463 306399 -64 - Misses 63813 63844 +31 ```
jeffwashington commented 1 week ago

got pulled in as part of #1420