Skip to content

perf(l1): (testing) try removing bloom filter to see if improves perf #5248

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
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

perf(l1): (testing) try removing bloom filter to see if improves perf #5248

wants to merge 1 commit into from
+0 −98

Conversation

@edg-l
Copy link
Contributor

@edg-l edg-l commented Nov 10, 2025

Motivation

Description

Closes #issue_number

@github-actions github-actions bot added L1 Ethereum client performance labels Nov 10, 2025
@github-actions
Copy link

github-actions bot commented Nov 10, 2025

Lines of code report

Total lines added: 0
Total lines removed: 61
Total lines changed: 61

Detailed view 
+-------------------------------------------+-------+------+
| File                                      | Lines | Diff |
+-------------------------------------------+-------+------+
| ethrex/crates/storage/trie_db/layering.rs | 123   | -61  |
+-------------------------------------------+-------+------+

@edg-l
Copy link
Contributor Author

edg-l commented Nov 10, 2025

Note: i ran it on notif-12 mainnet, it didnt seem to increase perf and went down to about 140mgas/s

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

Reviewers

No reviews

Assignees

@edg-l edg-l

Labels

L1 Ethereum client performance

Projects

ethrex_l1
Status: No status
ethrex_performance
Status: Todo

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@edg-l