171
72%

Ran 06 Aug 2018 03:14PM UTC

Jobs 1

Files 430

Run time 4min

Badge

Embed ▾

pending completion

Build # 171

Build Type

push

jenkins

Committed by Ivan Kelly

Commit Message

Make each ensemble in ensemble list immutable

Previously, the ensemble list was a Map<Long, ArrayList<BookieSocketAddress>>.
ArrayList is by definition mutable, so ensemble passed to metadata users are
always mutable.

This patch changes in ensembles in the list to be immutable. We were also leaking
the implementation of ledger metadata to the placement policy, so this has been
modified to use List<BookieSocketAddress> also.

Master issue: #281

Author: Ivan Kelly <ivan@ivankelly.net>

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

This closes #1575 from ivankelly/ledger-fragment-immutable-metadata and squashes the following commits:

14977a242 [Ivan Kelly] fix dlog
bdaef31e8 [Ivan Kelly] checkstyle
6e3431118 [Ivan Kelly] Make each ensemble in ensemble list immutable

Run Details

24660 of 34180 relevant lines covered (72.15%)

0.72 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	171.1	06 Aug 2018 03:14PM UTC	0	72.15

Source Files on build 171

Detailed source file information is not available for this build.

apache / bookkeeper / 171
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 171

apache / bookkeeper / 171 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 171

apache / bookkeeper / 171
72%

README BADGES
x