14287
82%
master: 87%

Ran 24 Oct 2018 11:26AM UTC

Jobs 1

Files 304

Run time 18s

Badge

Embed ▾

pending completion

Build # 14287

Build Type

push

travis-ci

Committed by

locker

Commit Message

tuple: zap tuple_extra

tuple_extra() allows to store arbitrary metadata inside tuples.
To use it, one should set extra_size when creating a tuple_format.
It was introduced for storing UPSERT counter or column mask inside
vinyl statements. Turned out that it wasn't really needed as UPSERT
counter can be stored on lsregion while column mask doesn't need to
be stored at all.

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().

So this patch gets rid of tuple_extra(). To do that, it partially
reverts the following commits:

6c0842e0302c vinyl: refactor vy_stmt_alloc()
74ff46d8e93d vinyl: add special format for tuples with column mask
11eb7816a30e Add extra size to tuple_format->field_map_size

Run Details

40872 of 49660 relevant lines covered (82.3%)

840616.46 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
5	14287.5 (TARGET=coverage)	24 Oct 2018 11:26AM UTC	304	82.3	Travis Job 14287.5

tarantool / tarantool / 14287
82%
master: 87%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 14287

tarantool / tarantool / 14287 82% master: 87%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 14287

tarantool / tarantool / 14287
82%
master: 87%

README BADGES
x