Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15409 )
Change subject: [build] Fail the LINT build with Java static analysis issues ...................................................................... [build] Fail the LINT build with Java static analysis issues This patch changes the Gradle build to fail when static analysis issues are found. In the past warnings would be logged and reports were created, but the build would still succeed. A `-PignoreCheckFailures` was added in case users wanted to manually revert back to the old behavior. Additionally the LINT Jenkins build was updated to run the static analysis checks and fail if any are found. Change-Id: I6fb8494639b1bbf9be6d2bd214a986b101496525 Reviewed-on: http://gerrit.cloudera.org:8080/15409 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M build-support/jenkins/build-and-test.sh M java/gradle/quality.gradle 2 files changed, 45 insertions(+), 8 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved Alexey Serbin: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/15409 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6fb8494639b1bbf9be6d2bd214a986b101496525 Gerrit-Change-Number: 15409 Gerrit-PatchSet: 8 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)
