Hello Tidy Bot, Attila Bukor, Kudu Jenkins, Andrew Wong, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14425
to look at the new patch set (#5).
Change subject: KUDU-2971 p1: add subprocess module
......................................................................
KUDU-2971 p1: add subprocess module
Utility classes exist that allow for IPC over stdin/stdout via protobuf
and JSON-encoded protobuf. This commit moves those classes into their
own directory so it can be reused by other subprocesses.
Following commits can then extend it to support concurrent communications
with subprocess. There are no functional changes in this patch.
Change-Id: If73e27772e1897a04f04229c4906a24c61e361f2
---
M CMakeLists.txt
A src/kudu/subprocess/CMakeLists.txt
A src/kudu/subprocess/subprocess_protocol.cc
A src/kudu/subprocess/subprocess_protocol.h
M src/kudu/tools/CMakeLists.txt
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_common.cc
M src/kudu/tools/tool_action_common.h
M src/kudu/tools/tool_action_test.cc
9 files changed, 374 insertions(+), 265 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/25/14425/5
--
To view, visit http://gerrit.cloudera.org:8080/14425
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If73e27772e1897a04f04229c4906a24c61e361f2
Gerrit-Change-Number: 14425
Gerrit-PatchSet: 5
Gerrit-Owner: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
