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.

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

Remove deposit contract log filtering for full node after pectra #14163

Open
joshieDo opened this issue Feb 3, 2025 · 0 comments
Open

Remove deposit contract log filtering for full node after pectra #14163

joshieDo opened this issue Feb 3, 2025 · 0 comments
Labels
A-pruning Related to pruning or full node C-enhancement New feature or request E-prague Related to the prague network upgrade

Comments

@joshieDo
Copy link
Collaborator

joshieDo commented Feb 3, 2025

Describe the feature

Currently, storing deposit contract receipts is actually unnecessary for most CLs (eip4881), but after pectra (eip6110) it will be 100% unnecessary.

receipts: chain_spec
.deposit_contract()
.map(|contract| PruneMode::Before(contract.block))
.or(Some(PruneMode::Distance(MINIMUM_PRUNING_DISTANCE))),
account_history: Some(PruneMode::Distance(MINIMUM_PRUNING_DISTANCE)),
storage_history: Some(PruneMode::Distance(MINIMUM_PRUNING_DISTANCE)),
receipts_log_filter: ReceiptsLogPruneConfig(
chain_spec
.deposit_contract()
.map(|contract| (contract.address, PruneMode::Before(contract.block)))
.into_iter()
.collect(),

Additional context

No response

@joshieDo joshieDo added A-pruning Related to pruning or full node C-enhancement New feature or request E-prague Related to the prague network upgrade labels Feb 3, 2025
@joshieDo joshieDo changed the title Remove deposit contract logs filtering for full node after pectra Remove deposit contract log filtering for full node after pectra Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-pruning Related to pruning or full node C-enhancement New feature or request E-prague Related to the prague network upgrade
Projects
Status: Todo
Development

No branches or pull requests

1 participant