GitHub user ijuma opened a pull request:
https://github.com/apache/kafka/pull/3801
MINOR: Include response in request log
It's implemented such that there is no overhead if request logging is
disabled.
Also:
- Reduce metrics computation duplication in `updateRequestMetrics`
- Change a couple of log calls to use string interpolation instead of
`format`
- Fix a few compiler warnings related to unused imports and unused default
arguments.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ijuma/kafka log-response-in-request-log
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/kafka/pull/3801.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #3801
----
commit f3d3c7436e5049b833655797757af736b0d8f7a1
Author: Ismael Juma <ism...@juma.me.uk>
Date: 2017-09-06T12:23:44Z
MINOR: Add response to the request log
commit 529a6fdd2bb929c04abe76d9e2644ca1fcd47d85
Author: Ismael Juma <ism...@juma.me.uk>
Date: 2017-09-06T12:39:22Z
Attempt at reducing duplication in `updateRequestMetrics`
commit da2dd69c92ca44b44ca436d1bf682f79596c8c0d
Author: Ismael Juma <ism...@juma.me.uk>
Date: 2017-09-06T12:40:12Z
Use string interpolation instead of `format` in a couple of log calls
commit 200915466ca5647ffd20198d35790412ae90f5c9
Author: Ismael Juma <ism...@juma.me.uk>
Date: 2017-09-06T12:40:54Z
Fix some compiler warnings
----
---
