-
Couldn't load subscription status.
- Fork 21.5k
core: speed up header import #21967
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
core: speed up header import #21967
Conversation
3ba14c9 to
90be6be
Compare
consensus/ethash/consensus.go
Outdated
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.
Why remove this check ?
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.
b5aa900 to
6bce7c4
Compare
consensus/ethash/consensus.go
Outdated
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.
unix :)
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.
Pls fix utx->unix, othewise SGTM
d06d950 to
92a53c9
Compare


This is a follow up to #21471.
Charts and niceties can be found here: https://github.com/holiman/headerimport/blob/main/README.md
This PR implements the following modifications
Nowtime.Nowfor every single headerTLDR: With these changes, writing 1M headers goes down to
62sfor 1M headers, from106son (old) master, and80son PR 21471.