Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20193 )
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]> (cherry picked from commit 2e953e48f91b2d099f6c6f3f18921c74d2ecfeeb) Reviewed-on: http://gerrit.cloudera.org:8080/20193 Tested-by: Yingchun Lai <[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 Yingchun Lai: Verified -- To view, visit http://gerrit.cloudera.org:8080/20193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I6523fdaaca19ee089dbac52a7dedec8847926a6c Gerrit-Change-Number: 20193 Gerrit-PatchSet: 2 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]>
