Hello Yingchun Lai, Attila Bukor, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19516
to look at the new patch set (#3).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/16/19516/3
--
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: newpatchset
Gerrit-Change-Id: I9fe57bdeeb8a5515578e3feb7cacbbb0d7692e3e
Gerrit-Change-Number: 19516
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <[email protected]>
