Abhishek Chennaka has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/20180 )

Change subject: [subprocess] KUDU-3489: Support reading large messages through 
pipes
......................................................................

[subprocess] KUDU-3489: Support reading large messages through pipes

This patch enables the subprocess server to be able to read messages
larger than 1MB which was otherwise flaky by reading the input stream
messages until we encounter EOF. This issue is noticed when large sized
requests are made to the subprocess server and it fails in receiving
the complete messages.

In addition made a small log change to MessageIO.java to display the
exception message correctly.

Change-Id: I6523fdaaca19ee089dbac52a7dedec8847926a6c
Reviewed-on: http://gerrit.cloudera.org:8080/20180
Reviewed-by: Alexey Serbin <[email protected]>
Tested-by: Abhishek Chennaka <[email protected]>
---
M java/kudu-subprocess/src/main/java/org/apache/kudu/subprocess/MessageIO.java
M 
java/kudu-subprocess/src/main/java/org/apache/kudu/subprocess/MessageReader.java
M 
java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/echo/TestEchoSubprocess.java
M src/kudu/subprocess/subprocess_server-test.cc
4 files changed, 81 insertions(+), 29 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved
  Abhishek Chennaka: Verified

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I6523fdaaca19ee089dbac52a7dedec8847926a6c
Gerrit-Change-Number: 20180
Gerrit-PatchSet: 7
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)

Reply via email to