347
72%

Ran 01 Feb 2019 02:56PM UTC

Jobs 1

Files 494

Run time 34s

Badge

Embed ▾

pending completion

Build # 347

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

26353 of 36425 relevant lines covered (72.35%)

0.72 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	347.1	01 Feb 2019 02:56PM UTC	0	72.35

Source Files on build 347

Detailed source file information is not available for this build.

apache / bookkeeper / 347
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 347

apache / bookkeeper / 347 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 347

apache / bookkeeper / 347
72%

README BADGES
x