Implement the Prefix type for the new forest
#701
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
iamrecursion
added
no changelog
iamrecursion
force-pushed
the
forest-prefix-container
branch
2 times, most recently
from
fd62876 to
0ec04b6
Compare
iamrecursion
changed the title
Prefix typePrefix type for the new forest
5 tasks
iamrecursion
force-pushed
the
forest-storage-interface
branch
from
84056a0 to
5556cd8
Compare
iamrecursion
force-pushed
the
forest-prefix-container
branch
4 times, most recently
from
4f7e0cc to
cf9f2d9
Compare
iamrecursion
force-pushed
the
forest-storage-interface
branch
from
5556cd8 to
6ada5f6
Compare
iamrecursion
force-pushed
the
forest-prefix-container
branch
from
cf9f2d9 to
4a38503
Compare
iamrecursion
force-pushed
the
forest-storage-interface
branch
from
6ada5f6 to
1571406
Compare
iamrecursion
force-pushed
the
forest-prefix-container
branch
from
4a38503 to
b504714
Compare
iamrecursion
force-pushed
the
forest-storage-interface
branch
from
1571406 to
416bd71
Compare
This is a container for the in-memory portion of each tree in the forest, and provides functionality to rebuild the prefix at startup from provided restoration data.
iamrecursion
force-pushed
the
forest-prefix-container
branch
from
b504714 to
e68fe44
Compare
Collaborator
Author
|
Closing in favour of an upcoming PR with a different approach. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe your changes
This is a container for the in-memory portion of each tree in the forest, and provides functionality to rebuild the prefix at startup from provided restoration data.
Much like for #700, a few places in the current code are marked as
#[allow(dead_code)]. This is to prevent warnings from failing the build while still allowing the private code to be merged incrementally.I also argue that, as this is internal code, it does not require a changelog entry. Please let me know if you disagree.
Checklist before requesting a review
nextaccording to naming convention.