couchbase
/
moss
/
114
/
1
83%
master: 83%
master: 83%
|
Ran
|
Files
20
|
Run time
1s
|
Badge
Embed ▾
README BADGES
|
coverage: 85.162% (-0.02%) from 85.182%
push
travis-ci
steveyenMB-21318 - compactMaybe also counts incoming higher snapshot Alex Gyryk showed some profiling results from his performance tests, where the profiling was strangely showing that mossStore wasn't using the optimized iteratorSingle implementation, even though the mossStore was supposed to be fully compacted. That is, a fully compacted mossStore should have just a single segment, so the iteratorSingle should have been utilized. The previous commit added stats that showed there were sometimes 2 segments even after compaction. So, this commit fixes the compactMaybe() logic to also account for the segments that are part of the incoming higher-level Snapshot. Change-Id: I20ffd52a5 Reviewed-on: http://review.couchbase.org/68771 Reviewed-by: Pavel Paulau <pavel.paulau@gmail.com> Tested-by: Steve Yen <steve.yen@gmail.com>
2181 of 2561 relevant lines covered (85.16%)
9602.79 hits per line
