Conversation
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
bench sendnow dispatches transactions round-robin across every supplied RPC endpoint, giving the multi-validator benchmark an even submission distribution. Limit post-workload block-stat fetches to four concurrent requests (previously 32) to reduce RPC-tunnel pressure during report collection; transaction concurrency and the measured workload window are unchanged.Validated with all 114 bench-core tests, nightly formatting and Clippy, and the full GitHub checks. Multi-region benchmark attempt 3 completed both phases and report collection with exit code 0: 10 validators across five AWS regions, 100 GiB state, 90-second load windows, 50,000 target TPS, and 5,000 concurrent submissions. Both sides used this txgen revision and Tempo
d093829e40704cf2472a3ef18d4a85695e6ec71f; all ten feature processes were verified to include--disable-tx-gossip, and all ten baseline argument files omitted it.Gossip on → off in this single pair: on-chain TPS 8,240 → 7,928 (−3.8%), RPC submission failures 1,609 → 0, builder pool-fetch p50 7.2 → 0.8 ms, and block p99 1,427 → 1,468 ms. The first two attempts timed out during baseline block-stat collection; the original reference run had also encountered that failure before this change.
Prompted by: @mediocregopher