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

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

Build:
DEFAULT BRANCH: master
Ran 14 Apr 2018 02:08PM UTC
Files 404
Run time 12s
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

14 Apr 2018 02:08PM UTC coverage: 72.107% (+0.03%) from 72.073%
52.1

push

jenkins

Sijie Guo
Fallback to use `getZkServers` and `getZkLedgersPath` when resolving from metadata service uri

Descriptions of the changes in this PR:

*Problme*

Dlog tests are failing because dlog is using an external zookeeper client, where both `zkServers` and `metadataServiceUri`
are not set on client configuration. It will throw NPE when trying to resolve `zkServers` and `zkLedgersRootPath` from metadata service uri.

*Solution*

Fallback to use deprecated `getZkServers` and `getZkLedgersPath` when metadata service uri is null

Related Issues: #1336

Author: Sijie Guo <sijie@apache.org>

Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Jia Zhai <None>

This closes #1338 from sijie/fix_dlog_tests

23132 of 32080 relevant lines covered (72.11%)

0.72 hits per line

Source Files on job 52.1
  • Tree
  • List 0
  • Changed 33
  • Source Changed 1
  • Coverage Changed 33
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 52
  • c3e96a76 on github
  • Prev Job for on master (#51.1)
  • Next Job for on master (#53.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