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

tree: allow batching new leaf nodes #362

Closed
wants to merge 1 commit into from
Closed

Conversation

gballet
Copy link
Member

@gballet gballet commented Jun 7, 2023

See #349, looks like github got confused by some forced push.

Signed-off-by: Ignacio Hagopian <[email protected]>

tree: simplify batching of new leaves creation

Signed-off-by: Ignacio Hagopian <[email protected]>

tree: fix insert new leaves test and benchmark

Signed-off-by: Ignacio Hagopian <[email protected]>

remove comment

Signed-off-by: Ignacio Hagopian <[email protected]>

remove conversion file

Signed-off-by: Ignacio Hagopian <[email protected]>

remove unused method

Signed-off-by: Ignacio Hagopian <[email protected]>
@jsign
Copy link
Collaborator

jsign commented Jun 7, 2023

See #349, looks like github got confused by some forced push.

@gballet , how dare you!? 😄

@gballet gballet force-pushed the jsign/cowleaves2 branch 2 times, most recently from 7f90724 to 78547b4 Compare June 7, 2023 18:16
@gballet
Copy link
Member Author

gballet commented Jun 9, 2023

@jsign it was for an investigation, I swear! #363, however, is totally meant to annoy you 😁 Closing this one.

@gballet gballet closed this Jun 9, 2023
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

Successfully merging this pull request may close these issues.

2 participants