Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14425 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/14425 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- 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(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- 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: merged Gerrit-Change-Id: If73e27772e1897a04f04229c4906a24c61e361f2 Gerrit-Change-Number: 14425 Gerrit-PatchSet: 6 Gerrit-Owner: Hao Hao <[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)
