Skip to content
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.

Sign up for GitHub

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

Jump to bottom

New trie version #478

Merged
merged 10 commits into from
Mar 4, 2022
Merged

New trie version #478

merged 10 commits into from
Mar 4, 2022

Conversation

lamafab
Copy link
Contributor

@lamafab lamafab commented Feb 21, 2022 •
edited

Adds:

  • New trie node header format.
  • Note on how to calculate the Merkle proof with "node hashes".
  • state_version field in Core_version runtime function.
  • ext_storage_root_version_2 Host API function.

@lamafab
Copy link
Contributor Author

lamafab commented Mar 3, 2022 •
edited

Note that this must be extended the the child_storage and trie too.

@lamafab lamafab mentioned this pull request Mar 4, 2022
Closed
FlorianFranzen
FlorianFranzen approved these changes Mar 4, 2022
View reviewed changes
Copy link
Contributor

@FlorianFranzen FlorianFranzen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will probably split the previous and the new extension header before merging into master, to be able to point from it from the changelog.

@FlorianFranzen FlorianFranzen merged commit 00236b9 into develop Mar 4, 2022
@FlorianFranzen FlorianFranzen deleted the new-trie-version branch March 4, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FlorianFranzen FlorianFranzen FlorianFranzen approved these changes

@drskalman drskalman Awaiting requested review from drskalman

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@lamafab @FlorianFranzen