Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/13297 )

Change subject: [build] Fix Gradle Checkstyle tasks
......................................................................

[build] Fix Gradle Checkstyle tasks

Checkstyle was broken in a recent upgrade due to
backwards incompatible changes to Checkstyle rules.

This patch adjust the Checkstyle task to be a bit more
Gradle idiomatic and also updates the checkstyle rules
to be more current.

Change-Id: I8f4d3aac746240949a32c798e27cd708a77966a4
Reviewed-on: http://gerrit.cloudera.org:8080/13297
Reviewed-by: Adar Dembo <[email protected]>
Tested-by: Kudu Jenkins
---
R java/config/checkstyle/checkstyle.xml
R java/config/checkstyle/suppressions.xml
M java/gradle/quality.gradle
3 files changed, 69 insertions(+), 30 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/13297
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8f4d3aac746240949a32c798e27cd708a77966a4
Gerrit-Change-Number: 13297
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to