• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

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

Build:
DEFAULT BRANCH: master
Ran 01 Aug 2018 02:51PM UTC
Files 430
Run time 5min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

01 Aug 2018 02:51PM UTC coverage: 72.168% (-0.02%) from 72.184%
162.1

push

jenkins

Ivan Kelly
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

24667 of 34180 relevant lines covered (72.17%)

0.72 hits per line

Source Files on job 162.1
  • Tree
  • List 0
  • Changed 46
  • Source Changed 30
  • Coverage Changed 32
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 162
  • 2b14458d on github
  • Prev Job for on master (#161.1)
  • Next Job for on master (#163.1)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc