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

Feat: Use in-memory binary Merkle tree #459

Merged
merged 1 commit into from
Jul 4, 2022
Merged

Feat: Use in-memory binary Merkle tree #459

merged 1 commit into from
Jul 4, 2022

Conversation

bvrooman
Copy link
Contributor

@bvrooman bvrooman commented Jul 4, 2022 •
edited
Loading

  • Updates fuel-merkle to v0.3.0
  • Replaces the usage of the standard binary Merkle tree and storage map with an in-memory binary Merkle tree
  • Uses the in-memory BMT's infallible interface and removes calls to expect

@bvrooman bvrooman self-assigned this Jul 4, 2022
@bvrooman bvrooman added enhancement New feature or request fuel-core-interfaces labels Jul 4, 2022
@bvrooman bvrooman marked this pull request as ready for review July 4, 2022 16:42
@bvrooman bvrooman merged commit 30e9c69 into master Jul 4, 2022
@bvrooman bvrooman deleted the bvrooman-feat-use-in-memory-binary-merkle-tree branch July 4, 2022 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Voxelot Voxelot Voxelot approved these changes

@vlopes11 vlopes11 Awaiting requested review from vlopes11

@rakita rakita Awaiting requested review from rakita

@adlerjohn adlerjohn Awaiting requested review from adlerjohn

@Dentosal Dentosal Awaiting requested review from Dentosal Dentosal is a code owner

@leviathanbeak leviathanbeak Awaiting requested review from leviathanbeak

Assignees

@bvrooman bvrooman

Labels
enhancement New feature or request fuel-core-interfaces
Projects
Fuel Network
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bvrooman @Voxelot