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.

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

Rebalancing tree #6

Open
domwoe opened this issue Mar 24, 2023 · 1 comment
Open

Rebalancing tree #6

domwoe opened this issue Mar 24, 2023 · 1 comment

Comments

@domwoe
Copy link

domwoe commented Mar 24, 2023

Hey @nomeata,

do you plan to add an implementation of a rebalancing tree as is implemented in the Rust ic-certified-map library or are there reasons you think this is not needed?
See the issue here.

@nomeata
Copy link
Owner

nomeata commented Mar 24, 2023

I believe it's not needed, because it's a trie that branches on the bits of the key which for uniformly distributed keys (as likely the case for example with hashes as labels) nicely yields a balanced tree. And even an unbalanced tree can't be deeper than the twice the number of bits in the key.

But it's true that the module should have some documentation on that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants