-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Improved eth_syncing
#10398
Improved eth_syncing
#10398
Conversation
So uum, I also fixed #10307 since I was at it, but forgot to make a new branch :P |
@Giulio2002 why updaing this value only in InsertBlocks is not enough? why need update in UFC also? |
Not sure if this is a good heuristics, I thought that perhaps on other chains, one day old blocks might get imported too? I am just not sure what we need in the future and at first glance ValidateChain and Forkchoice sounded like better candidates because they MUST be called near chain tip |
The approach is simple, every time the
Execution
module sees a block it saves the number in DB. That number is the heuristics for the "tip of the chain".Also changes from stages.Finish to stages.Execution. the issue was that for syncing up everything (stages.Headers as well) is in a single mdbx tx. Now we have separate key which is committed in a small
Update
before the processing.