add get_multiple_accounts to BenchTpsClient #27379
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
It turned out that requesting durable nonce blockhashes one by one is too expensive.
By requesting 100 of them (maximum possible), tps is x3 higher.
These changes will be used for #27151
In particular, at this line:
solana/bench-tps/src/bench.rs
Line 549 in 19de07a
Summary of Changes
get_multiple_accounts
to BenchTpsClient`BankClient
because neitherSyncClient
norAsyncClient
doesn't have it