Andrew Wong has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/15344 )
Change subject: subprocess: plumb Java metrics into C++
......................................................................
subprocess: plumb Java metrics into C++
This plumbs metrics from the Java subprocess into C++ and encapsulates
common code used to interact with a SubprocessServer bits (including the
metrics parsing) into the new SubprocessProxy template class.
This template is specialized for Echo{Request,Response}PB messages as
the new test-only EchoSubprocess, and adds echo-specific histogram
metrics based on those returned by the Echo Java subprocess.
Change-Id: I7260ea13717dfd4af0138f77dfb6e5d239b3bee2
Reviewed-on: http://gerrit.cloudera.org:8080/15344
Tested-by: Kudu Jenkins
Reviewed-by: Hao Hao <[email protected]>
Reviewed-by: Adar Dembo <[email protected]>
---
M src/kudu/subprocess/CMakeLists.txt
A src/kudu/subprocess/subprocess_proxy-test.cc
A src/kudu/subprocess/subprocess_proxy.h
3 files changed, 270 insertions(+), 0 deletions(-)
Approvals:
Kudu Jenkins: Verified
Hao Hao: Looks good to me, approved
Adar Dembo: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/15344
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I7260ea13717dfd4af0138f77dfb6e5d239b3bee2
Gerrit-Change-Number: 15344
Gerrit-PatchSet: 6
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
