Re: Detoast iterators -take 2

Hi Nikita,

Thanks for working on this.

> master:
> [...]
> Time: 47.346 ms
>
> patched:
> [...]
> Time: 7.607 ms

Looks impressive. I believe there is still some room for improvement though:

1. IMO the new API (create_detoast_iterator et al) should have
corresponding unit tests.
2. Suggestion: perhaps (entity)_(action) naming would be better, that
is: detoast_iterator_create(), detoast_iterator_free() etc.
3. If you could demonstrate that there is no performance degradation
for other scenarios that would be great.

-- 
Best regards,
Aleksander Alekseev