-
Notifications
You must be signed in to change notification settings - Fork 2.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
state sync: tune request timeout and chunkers #6566
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6566 +/- ##
==========================================
- Coverage 61.22% 61.13% -0.10%
==========================================
Files 295 295
Lines 27914 27932 +18
==========================================
- Hits 17091 17076 -15
- Misses 9111 9135 +24
- Partials 1712 1721 +9
|
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.
👍
I introduced a similar concept with reverse sync as I modelled it off how chunks are sent and received so there's also 4 blockFetchers
and a timeout of 10 seconds. Do you think there's any need to make these configurable.
Yeah, I think so. If you feel it's reasonable 👍 |
We could use the same config i.e. amount of |
Just as a side thought, the priority of chunks is 1 (lowest possible). Would it make sense, if there are networks who want to have specific seed nodes for providing state that they are also able to change the priority here. I have a feeling - although not properly tested - that the occasional failures we see arise because the peer receives the request and then prepares the chunk but then the envelope gets dropped before being sent |
(cherry picked from commit 7d961b5) # Conflicts: # CHANGELOG_PENDING.md # config/config.go # internal/statesync/reactor.go # internal/statesync/reactor_test.go # node/node.go # statesync/syncer.go
* state sync: tune request timeout and chunkers (#6566) (cherry picked from commit 7d961b5) # Conflicts: # CHANGELOG_PENDING.md # config/config.go # internal/statesync/reactor.go # internal/statesync/reactor_test.go # node/node.go # statesync/syncer.go * fix build * fix config * fix config Co-authored-by: Aleksandr Bezobchuk <alexanderbez@users.noreply.github.com> Co-authored-by: Aleksandr Bezobchuk <aleks.bezobchuk@gmail.com>
closes: #6564