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

apache / bookkeeper / 211
72%

Build:
DEFAULT BRANCH: master
Ran 17 Sep 2018 03:08PM UTC
Jobs 1
Files 439
Run time 2min
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
211

push

jenkins

Sijie Guo
Issue #850: Added request context across client and bookies.

Descriptions of the changes in this PR:

 - MDC context is passed to runnables, callables etc.
 - protocol extended, context is sent to bookie servers, restored there and back on client with the response.
   Hopefully did not miss some nuance on the server side, largely rely on changes in ordered executors to do all the magic.
 - did microbenchmarking of the protocol changes (strings added to protobuf, MDC context preserved/restored). Looks ok.
 - added unit tests.

(bug W-5291641) 
(bug W-5291648)


### Motivation

Troubleshooting of request-level failures/errors can be simplified if request id was passed through all the stages of the request, from threadpools on the client to bookies to the response back on the client. 
Log4j/Slf4j allows logging of MDC data so it makes sense to use this functionality for logging.

### Changes

 - MDC context is passed to runnables, callables etc.
 - protocol extended, context is sent to bookie servers, restored there and back on client with the response.
   Hopefully did not miss some nuance on the server side, largely rely on changes in ordered executors to do all the magic.
 - did microbenchmarking of the protocol changes (strings added to protobuf, MDC context preserved/restored). Looks ok.
 - added unit tests.

Master Issue: #850 




Author: Andrey Yegorov <ayegorov@salesforce.com>

Reviewers: Enrico Olivelli <eolivelli@gmail.com>, Jia Zhai <None>, Venkateswararao Jujjuri (JV) <None>, Sijie Guo <sijie@apache.org>

This closes #1672 from dlg99/feature/correlation_id, closes #850

161 of 161 new or added lines in 8 files covered. (100.0%)

24919 of 34569 relevant lines covered (72.08%)

0.72 hits per line

Jobs
ID Job ID Ran Files Coverage
1 211.1 17 Sep 2018 03:08PM UTC 0
72.08
Source Files on build 211
Detailed source file information is not available for this build.
  • Back to Repo
  • Jenkins Build #211
  • c9dc301f on github
  • Prev Build on master (#210)
  • Next Build on master (#212)
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

© 2025 Coveralls, Inc