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

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

Build:
DEFAULT BRANCH: master
Ran 20 May 2020 03:31PM UTC
Files 538
Run time 1min
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

20 May 2020 03:31PM UTC coverage: 71.701% (-0.1%) from 71.828%
819.1

push

jenkins

GitHub
[BOOKKEEPER-TOOL] Fix update rate-limiting for update-ledger command

### Motivation
Right now, `UpdateLedgerCmd` provides option `updatespersec` to throttle number of writes on zk. However, it throttles number of reads instead writes. Because of that it takes long time to complete this command as it slows down the read instead applying throttling while writing to zk.

### Modification
- Apply throttling while updating zk.
- If writes are being throttled then we also want to avoid accumulating reads so, added `maxOutstandingReads` option to manage max concurrent reads.

**Note:** We also need this change for #2321 .. so, I will rebase #2321 once this PR is merged.

Reviewers: Enrico Olivelli <eolivelli@gmail.com>

This closes #2336 from rdhabalia/update_rate

28271 of 39429 relevant lines covered (71.7%)

0.72 hits per line

Source Files on job 819.1
  • Tree
  • List 0
  • Changed 18
  • Source Changed 0
  • Coverage Changed 18
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 819
  • e9021a60 on github
  • Prev Job for on master (#818.1)
  • Next Job for on master (#822.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