346
72%

Ran 31 Jan 2019 03:04PM UTC

Jobs 1

Files 494

Run time 1min

Badge

Embed ▾

pending completion

Build # 346

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

Jobs

ID	Job ID	Ran	Files	Coverage
1	346.1	31 Jan 2019 03:04PM UTC	0	72.36

Source Files on build 346

Detailed source file information is not available for this build.

apache / bookkeeper / 346
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 346

apache / bookkeeper / 346 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 346

apache / bookkeeper / 346
72%

README BADGES
x