-
Notifications
You must be signed in to change notification settings - Fork 74
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
Implement downloading the chain storage #2876
base: main
Are you sure you want to change the base?
Conversation
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.
Automatically approving tomaka's pull requests. This auto-approval will be removed once more maintainers are active.
Another thing to note is that at the moment we use the "no proof" version of the network protocol. |
twiggy diff reportDifference in .wasm size before and after this pull request.
|
Doesn't work due to the lack of child tries support. |
I'm going to update this PR to use the "proof version" of the request, and afterwards I would like to merge it. |
cc #165
Adds a mode to the
warp_sync
where, instead of downloading just the runtime and then the consensus-related information, we download the entire storage of the chain.To achieve this, the
StorageGet
requests have been renamed toStorageGetKeys
, and a newStorageGetRange
request has been added.Works for Westend. For Polkadot I'm getting
UnexpectedEntriesCount
errors which I can't manage to solve at the moment.This needs to be solved, and this PR needs a bit of clean up, before being ready for review/merging. However it is 90% ready.