610
72%

Ran 23 Oct 2019 03:35PM UTC

Jobs 1

Files 539

Run time 1min

Badge

Embed ▾

pending completion

Build # 610

Build Type

push

jenkins

Committed by GitHub

Commit Message

[DLOG] LogReader shouldn't be added to pending if not locking

The pendingReaders set in BKDistributedLogManager exists so that if
the manager is closed which the lock is being acquired for a reader,
that reader will be closed (even though it hasn't been returned to the
client).

In the case that the reader is opened without a lock, there
is not async action being performed. Previously we were also adding
these readers to the pendingReaders, but they were never being removed
from the pendingReaders, causing a memory leak. This change avoids
adding no-locking readers to pendingReaders.


Reviewers: Enrico Olivelli <eolivelli@gmail.com>

This closes #2185 from ivankelly/orphan-reader

Run Details

28189 of 39320 relevant lines covered (71.69%)

0.72 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	610.1	23 Oct 2019 03:36PM UTC	0	71.69

Source Files on build 610

Detailed source file information is not available for this build.

apache / bookkeeper / 610
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 610

apache / bookkeeper / 610 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 610

apache / bookkeeper / 610
72%

README BADGES
x