1
72%
master: 72%

Ran 31 Jan 2019 03:04PM UTC

Files 494

Run time 57s

Badge

Embed ▾

Committed 31 Jan 2019 03:04PM UTC coverage: 72.36% (+0.02%) from 72.341%

Job # 346.1

Build Type

push

jenkins

Committed by Jia Zhai

Commit Message

Ignore usage of localNode in PlacementPolicy.


Descriptions of the changes in this PR:

'ignoreLocalNodeInPlacementPolicy' specifies whether to ignore localnode
in the internal logic of placement policy. If it is not possible or useful
to use Bookkeeper client node's (or AutoReplicator) rack/region info. for
placement policy then it is better to ignore localnode instead of false
alarming with error/warn log lines and metrics.

It is not valid to expect rack mapping information to be available for Bookkeeper
client and ReplicationWorker nodes. So if rackmapping info. is not available
then it is better to ignore creation of BookieNode by resolving rack information
for the client localnode. Otherwise it would log unnecessary warn/error log lines
and rack resolution failure metrics.

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

This closes #1917 from reddycharan/ignorelocalnodeinplacementpolicy

Run Details

26357 of 36425 relevant lines covered (72.36%)

0.72 hits per line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 346.1

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 346.1

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

README BADGES
x