-
Notifications
You must be signed in to change notification settings - Fork 640
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
Sync primary network checkpoints during bootstrapping #2752
Conversation
Co-authored-by: Stephen <stephen@avalabs.org>
@@ -446,7 +446,7 @@ func (i *ipTracker) resetBloom() error { | |||
return err | |||
} | |||
|
|||
count := max(maxIPEntriesPerValidator*i.gossipableIDs.Len(), minCountEstimate) | |||
count := max(maxIPEntriesPerNode*i.trackedIDs.Len(), minCountEstimate) |
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 is a minor bug fix. Previously we weren't accounting for manuallyTracked
nodes correctly in the bloom filter size. This caused frequent bloom filter resets once we were tracking manually tracking significantly more nodes than we were previously.
See added test.
"zGhQDuTyFxZrY3NE3QzBU4v6r8RphEJfMPUgZ2cNHuC7xHbcB" | ||
] | ||
} | ||
} |
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.
nit: missing newline at EOF (or was this done intentionally if it doesn't play nice w/ goembed?)
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 file was auto-generated. If we want to add a newline here, we should do it in the script.
Why this should be merged
This is a minimally invasive PR to increase the parallelization of fetching primary network blockchains.
How this works
By including periodic checkpoints of the primary network chains, we can fetch each chain segment in parallel rather than needing to fetch the whole chain serially.
How this was tested