Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/19516 )
Change subject: KUDU-3450 handling of oversized messages in Ranger process wrapper ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/19516/2/java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/TestMessageIO.java File java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/TestMessageIO.java: http://gerrit.cloudera.org:8080/#/c/19516/2/java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/TestMessageIO.java@158 PS2, Line 158: Assert.fail(String.format("unexpected exception: %s", e.getMessage())); why not just add throws Exception to the signature? -- 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: comment Gerrit-Change-Id: I9fe57bdeeb8a5515578e3feb7cacbbb0d7692e3e Gerrit-Change-Number: 19516 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Tue, 28 Feb 2023 14:36:51 +0000 Gerrit-HasComments: Yes
