Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13304 )
Change subject: [java] Upgrade to log4j 2 ...................................................................... Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/13304/2/java/kudu-test-utils/src/main/java/org/apache/kudu/test/CapturingToFileLogAppender.java File java/kudu-test-utils/src/main/java/org/apache/kudu/test/CapturingToFileLogAppender.java: http://gerrit.cloudera.org:8080/#/c/13304/2/java/kudu-test-utils/src/main/java/org/apache/kudu/test/CapturingToFileLogAppender.java@77 PS2, Line 77: // BufferedWriter to buffer up character conversions. > I guess there's also an issue with the old code if the BufferedWriter const The BufferedWriter does throw any checked exceptions. http://gerrit.cloudera.org:8080/#/c/13304/2/java/kudu-test-utils/src/main/java/org/apache/kudu/test/CapturingToFileLogAppender.java@87 PS2, Line 87: if (useGzip) { > If this throws, where do we close 'os' (i.e. the FileOutputStream created o Done -- To view, visit http://gerrit.cloudera.org:8080/13304 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic4e14a3a25c36d5f47d07f06c1c0e68dad42bc66 Gerrit-Change-Number: 13304 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 14 May 2019 12:20:57 +0000 Gerrit-HasComments: Yes
