618
72%

Ran 01 Nov 2019 03:25PM UTC

Jobs 1

Files 539

Run time 34s

Badge

Embed ▾

pending completion

Build # 618

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

28279 of 39328 relevant lines covered (71.91%)

0.72 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	618.1	01 Nov 2019 03:25PM UTC	0	71.91

Source Files on build 618

Detailed source file information is not available for this build.

apache / bookkeeper / 618
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 618

apache / bookkeeper / 618 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 618

apache / bookkeeper / 618
72%

README BADGES
x