Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/22768 )
Change subject: [java] KUDU-3657: fix publishing ...................................................................... Patch Set 19: (2 comments) http://gerrit.cloudera.org:8080/#/c/22768/18/java/kudu-backup/build.gradle File java/kudu-backup/build.gradle: http://gerrit.cloudera.org:8080/#/c/22768/18/java/kudu-backup/build.gradle@38 PS18, Line 38: implementation libs.proto > BTW, in kudu-spark-tools and kudu-spark this is still listed as implementat I changed this back to implementation, no reason for this to be api http://gerrit.cloudera.org:8080/#/c/22768/18/java/kudu-hive/build.gradle File java/kudu-hive/build.gradle: http://gerrit.cloudera.org:8080/#/c/22768/18/java/kudu-hive/build.gradle@27 PS18, Line 27: testCompileOnly libs. > Just to double-check: do we indeed need to declare this transitive dependen good catch, indeed this is no longer required -- To view, visit http://gerrit.cloudera.org:8080/22768 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie045805a68bfcd16325e1e4dbb38653cefe3a4b0 Gerrit-Change-Number: 22768 Gerrit-PatchSet: 19 Gerrit-Owner: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[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: Thu, 22 May 2025 13:53:47 +0000 Gerrit-HasComments: Yes
