-
Notifications
You must be signed in to change notification settings - Fork 432
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
Fix/clique validation on initial sync #3764
Conversation
…adersSync in order to have full epoch and be able to validate snapshot.
@@ -423,6 +433,7 @@ private bool ShouldBeInStateNodesMode(Snapshot best) | |||
bool hasFastSyncBeenActive = best.Header >= PivotNumber; | |||
bool hasAnyPostPivotPeer = AnyPostPivotPeerKnown(best.PeerBlock); | |||
bool notInFastSync = !best.IsInFastSync; | |||
bool notNeedToWaitForHeaders = NotNeedToWaitForHeaders; |
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.
Is it needed for the State Sync in ShouldBeInStateNodesMode ?
Why can't we start immediately ?
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.
we need to DL and validate Head header in order to know which state root to download
…on_on_initial_sync # Conflicts: # src/Nethermind/Nethermind.Synchronization.Test/ParallelSync/MultiSyncModeSelectorTests.cs
Changes:
On Clique, makes header download prerequisite for further FastSync. This was an issue, where we Clique on FastSync couldn't validate blocks, because of missing Headers from last epoch.
Types of changes
What types of changes does your code introduce?
Testing
Requires testing
In case you checked yes, did you write tests??