Hao Hao has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15184 )

Change subject: subprocess: add an optional sleep to Echo service
......................................................................

subprocess: add an optional sleep to Echo service

I found this pretty helpful in testing the C++ subprocess.

Change-Id: Iabe9da298a1b71912ddc87f48834278d347d8514
Reviewed-on: http://gerrit.cloudera.org:8080/15184
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
Reviewed-by: Hao Hao <[email protected]>
---
M 
java/kudu-subprocess-echo/src/main/java/org/apache/kudu/subprocess/echo/EchoProtocolHandler.java
M src/kudu/subprocess/subprocess.proto
2 files changed, 10 insertions(+), 0 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Adar Dembo: Looks good to me, approved
  Hao Hao: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/15184
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Iabe9da298a1b71912ddc87f48834278d347d8514
Gerrit-Change-Number: 15184
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to