Skip to content

test(net/tx): add transaction fetcher benchmark harness#26446

Draft
figtracer wants to merge 1 commit into
paradigmxyz:txpool-fetcher-v2from
figtracer:fig/tx-fetch-bench
Draft

test(net/tx): add transaction fetcher benchmark harness#26446
figtracer wants to merge 1 commit into
paradigmxyz:txpool-fetcher-v2from
figtracer:fig/tx-fetch-bench

Conversation

@figtracer

Copy link
Copy Markdown
Collaborator

Summary

  • add a manager-level harness that drives transaction announcements through TransactionsManager
  • cover retry and import behavior through deterministic mock peer sessions
  • add disjoint and overlapping Criterion workloads with untimed equal-work assertions

This gives the legacy fetcher and fetcher2 the same correctness and profiling boundary without exposing either implementation or changing the production fetcher. The measured region excludes setup, teardown, and request-stat collection, and the workload fits the manager's default capacity so both implementations complete identical work.

Part of the incremental rollout tracked in #26410.

Validation

  • cargo +nightly fmt --all -- --check
  • cargo +nightly clippy -p reth-network --lib --benches --tests --all-features -- -D warnings
  • cargo test -p reth-network --lib --features test-utils (184 passed)
  • cargo bench -p reth-network --bench transaction_fetcher --features test-utils -- --test
  • the same test and benchmark workloads pass unchanged on fetcher2 prototype 4970010438

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant