1
72%
master: 72%

Ran 31 Oct 2019 02:57PM UTC

Files 539

Run time 48s

Badge

Embed ▾

Committed 31 Oct 2019 02:57PM UTC coverage: 71.794% (-0.04%) from 71.836%

Job # 617.1

Build Type

push

jenkins

Committed by Enrico Olivelli

Commit Message

Removed mutex contention on BufferedChannel

### Motivation

Contention on `BufferedChannel` between the journal thread and the ForceWriteThread was introduced in #1228.

`unpersistentBytes` is only used if `unpersistedBytesBound > 0`, which is not true by default. We shouldn't be paying the penalty if not needed.

Also `position` doesn't need to be `AtomicLong` since it's only updated while the mutex is taken. Using a volatile will have the same effect with less overhead.


Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Jia Zhai <zhaijia@apache.org>

This closes #2187 from merlimat/buffered-channel-contention

Run Details

28235 of 39328 relevant lines covered (71.79%)

0.72 hits per line

apache / bookkeeper / 617 / 1
72%
master: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 617.1

apache / bookkeeper / 617 / 1 72% master: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 617.1

apache / bookkeeper / 617 / 1
72%
master: 72%

README BADGES
x