Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

newPayload latency spikes with pre-warming enabled #14520

Open
Rjected opened this issue Feb 15, 2025 · 0 comments · May be fixed by #14500
Open

newPayload latency spikes with pre-warming enabled #14520

Rjected opened this issue Feb 15, 2025 · 0 comments · May be fixed by #14500
Labels
A-trie Related to Merkle Patricia Trie implementation C-perf A change motivated by improving speed, memory usage or disk footprint

Comments

@Rjected
Copy link
Member

Rjected commented Feb 15, 2025

When pre-warming is enabled, sometimes the newPayload latency grows to over 100ms:

Image

We can also see that it can take a long time to fetch proofs:

Image

And that there are many storage targets:

Image
@Rjected Rjected added A-trie Related to Merkle Patricia Trie implementation C-perf A change motivated by improving speed, memory usage or disk footprint labels Feb 15, 2025
@jenpaff jenpaff linked a pull request Feb 18, 2025 that will close this issue
perf(root): chunk state updates in state root task #14500
Draft
@jenpaff jenpaff removed a link to a pull request Feb 18, 2025
perf(root): chunk state updates in state root task #14500
Draft
@jenpaff jenpaff linked a pull request Feb 18, 2025 that will close this issue
perf(root): chunk state updates in state root task #14500
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-trie Related to Merkle Patricia Trie implementation C-perf A change motivated by improving speed, memory usage or disk footprint
Projects
Reth Tracker
Status: Todo
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

perf(root): chunk state updates in state root task paradigmxyz/reth
1 participant
@Rjected