Grant Henke has posted comments on this change. Change subject: Fix checkstyle suppressions in submodule builds ......................................................................
Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6060/3/java/kudu_style.xml File java/kudu_style.xml: Line 54 > if we're no longer using this file, we should be able to remove it, right? We are still using it, it's just passed directly from the checkstyle maven plugin via the suppressionsLocation property in the pom. -- To view, visit http://gerrit.cloudera.org:8080/6060 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I75c5ae0cb5fcc468d15139619c113d2a67b024f7 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
