[tarantool-patches] Re: [PATCH 8/8] tuple: zap tuple_extra

Konstantin Osipov kostja at tarantool.org
Tue Oct 23 10:42:11 MSK 2018


* Vladimir Davydov <vdavydov.dev at gmail.com> [18/10/15 10:15]:
> Actually, the whole idea of tuple_extra() is rather crooked: why
> would we need it if we can inherit struct tuple instead, as we do
> in case of memtx_tuple and vy_stmt? Accessing an inherited struct
> is much more convenient than using tuple_extra().

Because you don't want to inherit a tuple to store last access
time, or some information specific to functional index or row id.
That was the idea at least.

This patch and the entire patch set is OK to push, when we get to
implementing the above mentioned features we can consider putting
it back in again.
> 
> So this patch gets rid of tuple_extra(). To do that, it partially
> reverts the following commits:

-- 
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