Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/22374 )
Change subject: Upgrade Java dependencies ...................................................................... Patch Set 21: (2 comments) http://gerrit.cloudera.org:8080/#/c/22374/21/java/kudu-hive/build.gradle File java/kudu-hive/build.gradle: PS21: Comparing the base version and PS21, it seems there were no actual modifications in this file. If it's so indeed, maybe revert this file to its original form, so it's clear that this file didn't require any updates from its base version? http://gerrit.cloudera.org:8080/#/c/22374/21/java/kudu-test-utils/build.gradle File java/kudu-test-utils/build.gradle: http://gerrit.cloudera.org:8080/#/c/22374/21/java/kudu-test-utils/build.gradle@75 PS21, Line 75: exclude "META-INF/versions/9/module-info.class" Is this still necessary given there is a broader 'exclude' pattern in gradle/shadow.gradle ? exclude "META-INF/versions/9/**", "META-INF/versions/21/**" -- To view, visit http://gerrit.cloudera.org:8080/22374 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id1b43e3cc8228e94fbbd3085933cd62bf089e23d Gerrit-Change-Number: 22374 Gerrit-PatchSet: 21 Gerrit-Owner: Abhishek Chennaka <[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: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Mon, 31 Mar 2025 16:34:35 +0000 Gerrit-HasComments: Yes
