1
72%
master: 72%

Ran 28 Aug 2018 03:12PM UTC

Files 437

Run time 59s

Badge

Embed ▾

Committed 28 Aug 2018 03:12PM UTC coverage: 72.45% (+0.08%) from 72.373%

Job # 193.1

Build Type

push

jenkins

Committed by Sijie Guo

Commit Message

ISSUE #1623: ReadOnlyLedgerHandle: don't schedule monitorPendingAddOps()

The LedgerHandle constructor schedules an addEntryQuorumTimeout check
with the bk client scheduler. However, the only place this callback is
canceled is in the closeAsync (the one which returns a future, not to be
confused with asyncClose) method. asyncClose and close() both leak this
callback. Moreover, ReadOnlyLedgerHandle invokes the LedgerHandle
constructor and so also creates this callback, but it overrides close()
and asyncClose() without passing them through.

ReadOnlyLedgerHandle already overrides
initializeExplicitLacFlushPolicy() to avoid write specific state.  This
patch generalizes that hack to initializeWriteHandleState() and the
cleanup to tearDownWriteHandleState().  tearDownWriteHandleState() is
moved into doAsyncClose(), which appears to be called for closes in
general.

(rev cguttapalem)
(bug W-5362724)
Signed-off-by: Samuel Just <sjustsalesforce.com>

Author: Samuel Just <sjust@salesforce.com>

Reviewers: Ivan Kelly <ivank@apache.org>, Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>

This closes #1624 from athanatos/forupstream/wip-1623, closes #1623

Run Details

24851 of 34301 relevant lines covered (72.45%)

0.72 hits per line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 193.1

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 193.1

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

README BADGES
x