-
Notifications
You must be signed in to change notification settings - Fork 2.2k
feat: parallelize recovery #20169
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
feat: parallelize recovery #20169
Conversation
mattsse
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this makes a lot of sense and is way easier than what I previously attempted
smol suggestion and a doc nit
we might also want to take the avail cores into account here but unsure because rayon already handles the bridging based on available threads
mattsse
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Bumps Reth to include paradigmxyz/reth#20169 and paradigmxyz/reth#20164 --------- Co-authored-by: Arsenii Kulikov <[email protected]>
Co-authored-by: Alexey Shekhirin <[email protected]> Co-authored-by: Matthias Seitz <[email protected]>
No description provided.