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

apache / bookkeeper / 470
72%

Build:
DEFAULT BRANCH: master
Ran 04 Jun 2019 03:33PM UTC
Jobs 1
Files 535
Run time 33s
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

pending completion
470

push

jenkins

Charan Reddy Guttapalem
Entries must be acknowledged by bookies in multiple fault domains before being acknowledged to client

Descriptions of the changes in this PR:

Bookkeeper write logic makes sure that there are at least ackQuorumSize
number of successful writes before sending ack back to the client. In
many cases these may fall into the same fault domain. A mechanism to
force bookkeeper to make sure that there are acks from at least
minNumRacksPerWriteQuorum number of fault domains and a configuration
to enforce this.

Signed-off-by: Ankit Jain <jain.asalesforce.com>

Master Issue: #2095 

Reviewers: Charan Reddy Guttapalem <reddycharan18@gmail.com>, Enrico Olivelli <eolivelli@gmail.com>

This closes #2096 from ankit-j/ankit-j/enforceFragmentMultipleFaultDomainWrite and squashes the following commits:

c90fd5a3d [Ankit Jain] Addressing review comments
07deae673 [Ankit Jain] Addressing @reddycharan's review comments
14164e291 [Ankit Jain] Fixed spacing error in bk_server.yaml
22c8b3c03 [Ankit Jain] Updated testing.
917ed1c45 [Ankit Jain] Move readLock.unlock to finally block
78e0cd501 [Ankit Jain] Modify test to not use default rack for bookies
ca0bc3b8b [Ankit Jain] Entries must be acknowledged by bookies in multiple fault domains before being acknowledged to client
d35aa22ad [Charan Reddy Guttapalem] Move common placementpolicy components to TopologyAwareEnsemblePlacementPolicy.

27216 of 38128 relevant lines covered (71.38%)

0.71 hits per line

Jobs
ID Job ID Ran Files Coverage
1 470.1 04 Jun 2019 03:33PM UTC 0
71.38
Source Files on build 470
Detailed source file information is not available for this build.
  • Back to Repo
  • Jenkins Build #470
  • b88f4b81 on github
  • Prev Build on master (#469)
  • Next Build on master (#471)
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