8313
87%
master: 87%

Ran 15 Jun 2017 01:29PM UTC

Jobs 1

Files 228

Run time 18s

Badge

Embed ▾

pending completion

Build # 8313

Build Type

push

travis-ci

Committed by

locker

Commit Message

vinyl: fix dump bandwidth calculation

We compute dump bandwidth basing on the time it takes a run writing task
to complete. While it used to work when we didn't have data compression
and indexes didn't share in-memory tuples, today the logic behind dump
bandwidth calculation is completely flawed:

 - Due to data compression, the amount of memory we dump may be much
   greater than the amount of data we write to disk, in which case dump
   bandwidth will be underestimated.

 - If a space has several indexes, dumping it may result in writing more
   data than is actually stored in-memory, because tuples of the same
   space are shared among its indexes in-memory, but stored separately
   when written to disk. In this case, dump bandwidth will be
   overestimated.

This results in quota watermark being set incorrectly and, as a result,
either stalling transactions or dumping memory non-stop.

Obviously, to resolve both issues, we need to account memory freed per
unit of time instead of data written to disk. So this patch makes
vy_scheduler_trigger_dump() remember the time when dump was started and
vy_scheduler_complete_dump() update dump bandwidth basing on the amount
of memory dumped and the time dump took.

Run Details

29141 of 33345 relevant lines covered (87.39%)

1135248.74 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
4	8313.4 (TARGET=coverage)	15 Jun 2017 01:29PM UTC	228	87.39	Travis Job 8313.4

Source Files on build 8313

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

tarantool / tarantool / 8313
87%
master: 87%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 8313

tarantool / tarantool / 8313 87% master: 87%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 8313

tarantool / tarantool / 8313
87%
master: 87%

README BADGES
x