go-algorand
go-algorand copied to clipboard
prefetcher: enable prefetcher for assets and apps
Summary
Enable prefetcher for asset and app transactions except the foreign fields.
Test Plan
- Modified all prefetcher error tests to ensure all transaction groups returned by the prefetecher even in case errors.
- Checked prefetcher_alignment_test that validates prefetcher's behavior againsy
apply
logic.
Codecov Report
Merging #4352 (6bdb584) into master (7e8ca90) will increase coverage by
0.06%
. The diff coverage is67.64%
.
@@ Coverage Diff @@
## master #4352 +/- ##
==========================================
+ Coverage 54.11% 54.18% +0.06%
==========================================
Files 401 401
Lines 51549 51583 +34
==========================================
+ Hits 27898 27952 +54
+ Misses 21311 21282 -29
- Partials 2340 2349 +9
Impacted Files | Coverage Δ | |
---|---|---|
ledger/internal/eval.go | 67.15% <8.57%> (-0.69%) |
:arrow_down: |
ledger/internal/prefetcher/prefetcher.go | 95.51% <98.50%> (+27.32%) |
:arrow_up: |
ledger/roundlru.go | 90.56% <0.00%> (-5.67%) |
:arrow_down: |
network/wsPeer.go | 66.03% <0.00%> (-4.86%) |
:arrow_down: |
ledger/blockqueue.go | 85.63% <0.00%> (-2.88%) |
:arrow_down: |
crypto/merkletrie/trie.go | 66.42% <0.00%> (-2.19%) |
:arrow_down: |
crypto/merkletrie/node.go | 91.62% <0.00%> (-1.87%) |
:arrow_down: |
catchup/peerSelector.go | 98.95% <0.00%> (-1.05%) |
:arrow_down: |
catchup/service.go | 68.14% <0.00%> (-0.50%) |
:arrow_down: |
network/wsNetwork.go | 64.82% <0.00%> (ø) |
|
... and 3 more |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
ops, broke some asset receiver prefetching, fixing
Fixed one unit test and adjusted AssetReceiver prefetching condition.
Thank you for review! I'll merge it right after the boxes go through to prevent possible conflicts.