12425
83%
master: 87%

Ran 07 Jun 2018 03:23PM UTC

Jobs 1

Files 301

Run time 45s

Badge

Embed ▾

pending completion

Build # 12425

Build Type

push

travis-ci

Committed by

kostja

Commit Message

vinyl: do not yield on dump completion

The fact that we may yield after we added a new slice created by dump,
but before we removed the dumped in-memory index from the LSM tree
complicates read iterator logic, as it has to detect such a case and
filter out sources that contain duplicates. This logic relies on the
fact that no two slices of the same range intersect by LSN. For the
sake of ALTER we have to relax this limitation, as statements inserted
during index build can have arbitrary (not monotonically growing) LSNs,
so the no-LSN-intersection property won't be fulfilled for whole slices,
only for individual keys. Since there shouldn't be more than 1000 ranges
in the same LSM tree, yielding doesn't make much sense as iteration over
the whole range tree should be pretty fast. Besides, dump isn't done
frequently. That said, let's remove yielding altogether.

Needed for #1653

Run Details

39216 of 47985 relevant lines covered (81.73%)

888223.68 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
1	80.78	0.58%	src/box/xlog.c
1	97.5	-1.25%	src/histogram.c
1	88.76	-0.56%	src/box/tuple_extract_key.cc
3	88.11	-0.3%	src/box/xrow.c
3	76.27	-1.69%	src/lib/salad/mhash.h
3	88.52	-0.82%	src/box/applier.cc

Jobs

ID	Job ID	Ran	Files	Coverage
3	12425.3 (TARGET=coverage)	07 Jun 2018 03:23PM UTC	301	81.73	Travis Job 12425.3

tarantool / tarantool / 12425
83%
master: 87%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 12425

tarantool / tarantool / 12425 83% master: 87%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 12425

tarantool / tarantool / 12425
83%
master: 87%

README BADGES
x