Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/22589 )
Change subject: [java] Modernise codebase ...................................................................... Patch Set 1: > Patch Set 1: > > There are too many warnings and style errors in PS1. It usually a good idea > to run lint-related Gradle tasks locally before submitting a patch for review > if working on a code change like this. > > https://jenkins.kudu.apache.org/job/build_and_test/7120/console > > FAILURE: Build failed with an exception. > > * What went wrong: > Execution failed for task ':kudu-client:checkstyleTest'. > > A failure occurred while executing > > org.gradle.api.plugins.quality.internal.CheckstyleAction > > Checkstyle rule violations were found. See the report at: > file:///home/jenkins-slave/workspace/build_and_test/java/kudu-client/build/reports/checkstyle/test.html > Checkstyle files with violations: 5 > Checkstyle violations by severity: [warning:40] Yeah, I ran `./gradlew checkstyleMain` which passed, but apparently it doesn't check the test files, fixing these in the next PS. -- To view, visit http://gerrit.cloudera.org:8080/22589 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ice41e86c666e7589662ec0c5ffa0b0e7dd493a98 Gerrit-Change-Number: 22589 Gerrit-PatchSet: 1 Gerrit-Owner: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Fri, 07 Mar 2025 08:40:57 +0000 Gerrit-HasComments: No
