Will Berkeley has posted comments on this change. Change subject: [java] Add checkstyle coverage to Java build ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5016/1/java/kudu-flume-sink/src/main/java/org/apache/kudu/flume/sink/AvroKuduOperationsProducer.java File java/kudu-flume-sink/src/main/java/org/apache/kudu/flume/sink/AvroKuduOperationsProducer.java: Line 268: + "Specify configuration property '%s' or event header '%s'", > I think we're pretty strict about always having the operator before the lin Yup, I can toggle it to look for operators like + on previous line. -- To view, visit http://gerrit.cloudera.org:8080/5016 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1578371509f7fcaaffa63bec0795bb3cf56c03c6 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com> Gerrit-HasComments: Yes