Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14775 )
Change subject: [java] Fix kudu-client checkstyle warnings ...................................................................... Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/14775/3/java/kudu-client/src/main/java/org/apache/kudu/Type.java File java/kudu-client/src/main/java/org/apache/kudu/Type.java: http://gerrit.cloudera.org:8080/#/c/14775/3/java/kudu-client/src/main/java/org/apache/kudu/Type.java@59 PS3, Line 59: Private > nit: it's no longer private. Or it means package private? Private here is redundant. Enum constructors are private by default. http://gerrit.cloudera.org:8080/#/c/14775/3/java/kudu-client/src/main/java/org/apache/kudu/client/KeyRange.java File java/kudu-client/src/main/java/org/apache/kudu/client/KeyRange.java: http://gerrit.cloudera.org:8080/#/c/14775/3/java/kudu-client/src/main/java/org/apache/kudu/client/KeyRange.java@97 PS3, Line 97: "<start>" : Bytes.hex(primaryKeyStart), > Is this what checkstyle recommends? For some reason, I found it a bit hard Operators are supposed to be before the line break. http://gerrit.cloudera.org:8080/#/c/14775/3/java/kudu-client/src/main/java/org/apache/kudu/client/Negotiator.java File java/kudu-client/src/main/java/org/apache/kudu/client/Negotiator.java: http://gerrit.cloudera.org:8080/#/c/14775/3/java/kudu-client/src/main/java/org/apache/kudu/client/Negotiator.java@453 PS3, Line 453: saslCallback); > nit: any reason to have this is so out-of-bound compared with other argumen no, not sure how that happened. I will fix it in a follow up change. -- To view, visit http://gerrit.cloudera.org:8080/14775 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I153b7bf11b505871703da729720c988dd7c212f9 Gerrit-Change-Number: 14775 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 26 Nov 2019 14:04:24 +0000 Gerrit-HasComments: Yes
