Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14801 )
Change subject: [java] Enable and fix low SpotBugs warnings ...................................................................... Patch Set 2: Code-Review+2 (4 comments) http://gerrit.cloudera.org:8080/#/c/14801/2/java/kudu-client-tools/src/main/java/org/apache/kudu/mapreduce/tools/IntegrationTestBigLinkedList.java File java/kudu-client-tools/src/main/java/org/apache/kudu/mapreduce/tools/IntegrationTestBigLinkedList.java: http://gerrit.cloudera.org:8080/#/c/14801/2/java/kudu-client-tools/src/main/java/org/apache/kudu/mapreduce/tools/IntegrationTestBigLinkedList.java@739 PS2, Line 739: context.write(new Text(keyString), new Text(refsList.toString())); > No, refsList is guaranteed not to be null because it's initialized just abo Indeed. http://gerrit.cloudera.org:8080/#/c/14801/2/java/kudu-client/src/main/java/org/apache/kudu/client/PleaseThrottleException.java File java/kudu-client/src/main/java/org/apache/kudu/client/PleaseThrottleException.java: http://gerrit.cloudera.org:8080/#/c/14801/2/java/kudu-client/src/main/java/org/apache/kudu/client/PleaseThrottleException.java@69 PS2, Line 69: transient > Exceptions are marked as serializable, but these fields aren't serialized. Yep, that makes sense if we want to get rid of warnings. http://gerrit.cloudera.org:8080/#/c/14801/2/java/kudu-client/src/main/java/org/apache/kudu/client/RowResultIterator.java File java/kudu-client/src/main/java/org/apache/kudu/client/RowResultIterator.java: http://gerrit.cloudera.org:8080/#/c/14801/2/java/kudu-client/src/main/java/org/apache/kudu/client/RowResultIterator.java@116 PS2, Line 116: if (!hasNext()) { : throw new NoSuchElementException(); : } > We have decent coverage, but always called hasNext() before next() or used I meant we don't have tests to verify the iterator-related contracts for this next() method. Anyway, adding such a test is beyond the scope of this patch. http://gerrit.cloudera.org:8080/#/c/14801/2/java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java File java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java: http://gerrit.cloudera.org:8080/#/c/14801/2/java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java@437 PS2, Line 437: X509Certificate[0]; > SpotBugs reported "PZLA_PREFER_ZERO_LENGTH_ARRAYS: Consider returning a zer Thank you for the clarification. -- To view, visit http://gerrit.cloudera.org:8080/14801 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4e5c8285554ff84e948c0ea095ba476969377440 Gerrit-Change-Number: 14801 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 27 Nov 2019 18:10:06 +0000 Gerrit-HasComments: Yes
