Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19516 )
Change subject: KUDU-3450 handling of oversized messages in Ranger process wrapper ...................................................................... KUDU-3450 handling of oversized messages in Ranger process wrapper This patch updates the Ranger client process wrapper to handle oversized messages. With this patch, upon receiving an oversized message, the wrapper discards the message and logs about the error. The input stream is ready to receive next messages after that. A corresponding unit test for MessageIO.readBytes() is added as well. In addition, now the --subprocess_max_message_size_bytes master's flag controls the maximum message size at the Ranger client process wrapper as well. SubprocessConfiguration.MAX_MESSAGE_BYTES_DEFAULT has been updated to match the default value of the flag. This is a follow-up to ae22d32ef5895a02a763665cfd2812aa61be5ab3. Change-Id: I9fe57bdeeb8a5515578e3feb7cacbbb0d7692e3e Reviewed-on: http://gerrit.cloudera.org:8080/19516 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Attila Bukor <[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/main/java/org/apache/kudu/subprocess/SubprocessConfiguration.java M java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/TestMessageIO.java M src/kudu/ranger/ranger_client.cc 5 files changed, 79 insertions(+), 26 deletions(-) Approvals: Alexey Serbin: Verified Attila Bukor: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/19516 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9fe57bdeeb8a5515578e3feb7cacbbb0d7692e3e Gerrit-Change-Number: 19516 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]>
