find-id in the pool should be heavily cached. It's not, and that's a mistake.
Note that the pool only grows, so there is no need to cache by the pool root. However, if it ever returns nil then don't cache it, since the pool may have expanded. (or remember that the nil came from a pool root id)
find-id
in the pool should be heavily cached. It's not, and that's a mistake.Note that the pool only grows, so there is no need to cache by the pool root. However, if it ever returns
nil
then don't cache it, since the pool may have expanded. (or remember that thenil
came from a pool root id)The same should happen for
find-object