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

Respect Storage History prune part in the pruner #3694

Closed
Tracked by #3428
shekhirin opened this issue Jul 10, 2023 · 0 comments · Fixed by #4006
Closed
Tracked by #3428

Respect Storage History prune part in the pruner #3694

shekhirin opened this issue Jul 10, 2023 · 0 comments · Fixed by #4006
Labels
A-pruning Related to pruning or full node C-enhancement New feature or request
Milestone
0.1.0-alpha.5

Comments

@shekhirin
Copy link
Collaborator

shekhirin commented Jul 10, 2023 •
edited
Loading

Describe the feature

Prune StorageChangeSet and StorageHistoryIndex tables in the pruner.

If Account History prune target is less than 64 blocks, don't prune StorageChangeSet table until the target is > 64 blocks. StorageHistoryIndex can be safely pruned, as they are not needed for the unwind.

Additional context

No response
@shekhirin shekhirin mentioned this issue Jul 10, 2023
Closed
@shekhirin shekhirin added C-enhancement New feature or request A-pruning Related to pruning or full node labels Jul 10, 2023
@shekhirin shekhirin added this to the 0.1.0-alpha.4 milestone Jul 10, 2023
@onbjerg onbjerg modified the milestones: 0.1.0-alpha.4, 0.1.0-alpha.5 Jul 19, 2023
@shekhirin shekhirin mentioned this issue Jul 31, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-pruning Related to pruning or full node C-enhancement New feature or request
Projects
Reth Tracker
Archived in project
Milestone
0.1.0-alpha.5
Development

Successfully merging a pull request may close this issue.

feat(pruner): storage history paradigmxyz/reth
2 participants
@shekhirin @onbjerg