[tarantool-patches] Re: [PATCH 5/6] vinyl: run iterator: refactor seek method

Konstantin Osipov kostja at tarantool.org
Thu Mar 28 17:39:47 MSK 2019


* Vladimir Davydov <vdavydov.dev at gmail.com> [19/03/26 18:56]:
> A few changes to make the function more straightforward:
> 
>  - Move bloom checking and LSN filtering out of 'do_seek' helper. Make
>    the helper do just one simple task - lookup the first one in a series
>    of statements matching the given search criteria.
>  - Fold iterator type and key substitution in 'seek' method, similarly
>    to how we did it for other iterators.
>  - Cleanup EQ checks. Use the original iterator type and key where
>    appropriate to remove extra checks in calling methods. Don't check EQ
>    in 'seek' method in case it was checked by 'do_seek'.
>  - Add some comments.

LGTM (same comment re passing EOF in return value applies).
eq_found looks cumbersome, perhaps you could both kill bool
eq_found and not pass eof in the out parameter of do_seek()?


-- 
Konstantin Osipov, Moscow, Russia, +7 903 626 22 32
http://tarantool.io - www.twitter.com/kostja_osipov




More information about the Tarantool-patches mailing list