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

apache / bookkeeper / 192
72%

Build:
DEFAULT BRANCH: master
Ran 27 Aug 2018 03:12PM UTC
Jobs 1
Files 437
Run time 43s
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
192

push

jenkins

Sijie Guo
Provide async version of markLedgerUnderreplicated for LedgerUnderreplicationManager

Descriptions of the changes in this PR:

 ### Motivation

Auditor has multiple places calling sync methods in async callbacks.
This raises the possibility hitting deadlock. Issue #1578 is one of the examples.

After looking into the `LedgerUnderreplicationManager`, `markLedgerUnderreplicated`
is the only interface that will be called in async callbacks. This change is
to provide an async version of `markLedgerUnderreplicated`.

 ### Changes

- add `markLedgerUnderreplicatedAsync` interface in `LedgerUnderreplicationManager`.
- implement the logic of `markLedgerUnderreplicated` using async callbacks
- use `markLedgerUnderreplicatedAsync` in the Auditor

Related Issues: #1578
Master Issue: #1617

Author: Sijie Guo <sijie@apache.org>

Reviewers: Charan Reddy Guttapalem <reddycharan18@gmail.com>, Enrico Olivelli <eolivelli@gmail.com>, Matteo Merli <mmerli@apache.org>

This closes #1619 from sijie/async_sync_autorecovery

132 of 132 new or added lines in 5 files covered. (100.0%)

24824 of 34300 relevant lines covered (72.37%)

0.72 hits per line

Jobs
ID Job ID Ran Files Coverage
1 192.1 27 Aug 2018 03:12PM UTC 0
72.37
Source Files on build 192
Detailed source file information is not available for this build.
  • Back to Repo
  • Jenkins Build #192
  • 73b428c7 on github
  • Prev Build on master (#191)
  • Next Build on master (#193)
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