-
Notifications
You must be signed in to change notification settings - Fork 64
Closed
Description
Current union (addAll
) implementation for sets is O(logN * M) as far as I understand even when both sides have the same implementation.
It is certainly possible to improve that to log or even sub-log implementation for HAMTs by merging one-bit arrays only.
It is also possible to improve intersection, although atm you don't even have a method for intersection in the API.
Not sure if it helps maps in any way, but I guess it does.
While intersection is questionable, union is a very important and widely used operation (especially so when +
is overloaded as union) for persistent sets and implementing this doesn't even need changing the api.
Metadata
Metadata
Assignees
Labels
No labels