Skip to content

refactor(l1): unify fullsync_headers table and headers table #5267

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 3 commits into
base: main
Choose a base branch
from
Draft

refactor(l1): unify fullsync_headers table and headers table #5267

wants to merge 3 commits into from

Conversation

@cdiielsi
Copy link
Contributor

@cdiielsi cdiielsi commented Nov 11, 2025

Motivation

Description

Closes #4903

@github-actions github-actions bot added the L1 Ethereum client label Nov 11, 2025
@github-actions
Copy link

github-actions bot commented Nov 11, 2025 •
edited
Loading

Lines of code report

Total lines added: 24
Total lines removed: 27
Total lines changed: 51

Detailed view 
+---------------------------------------------+-------+------+
| File                                        | Lines | Diff |
+---------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/sync.rs        | 1369  | +1   |
+---------------------------------------------+-------+------+
| ethrex/crates/storage/api.rs                | 248   | +2   |
+---------------------------------------------+-------+------+
| ethrex/crates/storage/store.rs              | 1514  | -6   |
+---------------------------------------------+-------+------+
| ethrex/crates/storage/store_db/in_memory.rs | 651   | +21  |
+---------------------------------------------+-------+------+
| ethrex/crates/storage/store_db/rocksdb.rs   | 1574  | -21  |
+---------------------------------------------+-------+------+

cdiielsi
cdiielsi commented Nov 11, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@cdiielsi cdiielsi

Labels

L1 Ethereum client

Projects

ethrex_l1
Status: No status

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Unify headers and fullsync_headers tables

2 participants

@cdiielsi