5
83%
master: 88%

Ran 25 Jun 2019 01:07PM UTC

Files 305

Run time 24s

Badge

Embed ▾

Committed 25 Jun 2019 12:34PM UTC coverage: 82.491% (+0.1%) from 82.359%

Job # TARGET=coverage

Build Type

push

travis-ci

Committed by

locker

Commit Message

vinyl: free region on vylog commit instead of resetting it

region_reset() only frees memory from the last slab. As a result, if
a vylog transaction happens to use more than one slab, memory used by
vy_log.pool won't be freed, neither will it be reused, i.e. we'll get
a memory leak. Fix it by using region_free() instead of region_reset().
It's okay from performance point of view, because vylog transactions
are rare events.

Note, the master branch isn't affected to this issue, because the vylog
memory management was completely reworked there. 2.1 isn't affected
either, because there region_reset() was modified to free all slabs.
However, rather than backporting any of those commits, I think it's
more appropriate to simply use region_free().

Coverage Stats

41588 of 50415 relevant lines covered (82.49%)

844202.71 hits per line

tarantool / tarantool / 17579 / 5
83%
master: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 17579.5 (TARGET=coverage)

tarantool / tarantool / 17579 / 5 83% master: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 17579.5 (TARGET=coverage)

tarantool / tarantool / 17579 / 5
83%
master: 88%

README BADGES
x