[Tarantool-patches] [PATCH v3] memtx: fix out of memory handling for rtree

Konstantin Osipov kostja.osipov at gmail.com
Thu Dec 19 18:09:07 MSK 2019


* Olga Arkhangelskaia <arkholga at tarantool.org> [19/12/19 17:45]:
> I can't agree - everything that has memtx_ should be in memtx, because
> generic is used by vinyl, and sysview.

Neither vinyl nor sysview build secondary indexes using
build_next. This name - generic_* is a conversion mistake
introduced during C++ -> C transition, it should be
memtx_default_build_next. The generic build next should
assert(false), it should be never called. If, for whatever reason,
it's called, you still can split have a generic_memtx_build_next

> And the only way I see do it generic - is to use index_reserve? Did you mean
> index_reserve, that has  memtx_index_extent_reserve( inside?
> 
> In this case it is not shorter.
> 

-- 
Konstantin Osipov, Moscow, Russia


More information about the Tarantool-patches mailing list