Re: [Tarantool-patches] [PATCH v2] vinyl: restart read iterator in case of rolled back WAL

[Konstantin Osipov <kostja.osipov@gmail.com>]

* Aleksandr Lyapunov <alyapunov@tarantool.org> [20/06/23 08:16]:
> 
> > > > The read iterator was rewritten several times and still have at least
> > > > several bugs. I think we should admit that we cannot support such a
> > > > complicated solution. How about some stupid solution: if ANY change
> > > > has been happened during yield - restart advancing?
> > Does this statement have technical merit? Is it supported by
> > tests? I'd gladly support the change if it was grounded in reason -
> > evaluation of the performance impact, for example, could serve as
> > a confirmation that a simple solution would be just fine.
> > 
> > Without it, I'd it's a regress, a signature of helplessness,
> > lack of courage to make things right.
> Two reasons:
> 1. since we have to read from disk it's not a big deal to scan memory again.

Makes sense. Make sense now, when we yield only to read from disk.
In future we may consider yielding for other reasons, but this
looks like something rather distant..

> 2. a dozen of lines of code is removed - the cost of support is lowered a
> bit.



-- 
Konstantin Osipov, Moscow, Russia