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

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

Build:
DEFAULT BRANCH: master
Ran 19 May 2020 03:39PM 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

19 May 2020 03:39PM UTC coverage: 71.828% (-0.02%) from 71.844%
818.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

28321 of 39429 relevant lines covered (71.83%)

0.72 hits per line

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