Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/22768 )
Change subject: [java] KUDU-3657: fix publishing ...................................................................... Patch Set 10: > Patch Set 10: > > (1 comment) If you check the install/publish directory, you'll see that the kudu-proto.jar is not present. This is because we skip the publishing of it on purpose, as it's contents are shaded into the kudu-client.jar (see note at kudu-proto/build.gradle#40). The kudu-proto.jar will be generated when running a simple ./gradlew assemble, under '/kudu/java/kudu-proto/build/libs' you can check it's contents and the contents of the kudu-client.jar with 'jar -tf <jar_path>' which will show that the generated proto classes and the google protobuf classes are both present in the kudu-client jar, so the change of scope for this shouldn't cause any classpath issue. -- 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: 10 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: Tue, 29 Apr 2025 07:56:30 +0000 Gerrit-HasComments: No
