Skip to content

Conversation

@dhyaniarun1993
Copy link
Collaborator

Utility for #446

@dhyaniarun1993 dhyaniarun1993 self-assigned this Jan 6, 2026
@dhyaniarun1993 dhyaniarun1993 added K-perf Kind: performance K-bench Kind: benchmark labels Jan 6, 2026
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.34%. Comparing base (69f6061) to head (207d8d6).

Additional details and impacted files

Impacted file tree graph
see 2 files with indirect coverage changes

Flag Coverage Δ
e2e 35.74% <ø> (-0.01%) ⬇️
unit 26.89% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
reth binary 53.14% <ø> (ø)
op historical proof 90.12% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dhyaniarun1993 dhyaniarun1993 marked this pull request as ready for review January 7, 2026 08:54
Copilot AI review requested due to automatic review settings January 7, 2026 08:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a Go-based benchmark utility to measure the performance of the eth_getProof RPC method by testing historical proof requests against the WETH contract on Optimism. The utility is designed to support performance analysis for issue #446.

Key changes:

  • Implements a concurrent benchmark tool that measures eth_getProof latency across configurable block ranges
  • Targets WETH contract with pre-defined holder addresses and calculates storage slots for balance mappings
  • Reports aggregated latency statistics (avg/min/max) and error counts per block interval

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@emhane emhane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we pls move this to a new directory op-reth-bench in
https://github.com/op-rs/op-reth/tree/unstable/bin

Copy link
Collaborator

@itschaindev itschaindev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

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

Labels

K-bench Kind: benchmark K-perf Kind: performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants