619
72%

Ran 02 Nov 2019 03:20PM UTC

Jobs 1

Files 539

Run time 33s

Badge

Embed ▾

pending completion

Build # 619

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

28280 of 39328 relevant lines covered (71.91%)

0.72 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	619.1	02 Nov 2019 03:20PM UTC	539	71.91

apache / bookkeeper / 619
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 619

apache / bookkeeper / 619 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 619

apache / bookkeeper / 619
72%

README BADGES
x