Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/22374 )
Change subject: Upgrade Java dependencies ...................................................................... Patch Set 20: (2 comments) http://gerrit.cloudera.org:8080/#/c/22374/20/java/kudu-hive/build.gradle File java/kudu-hive/build.gradle: http://gerrit.cloudera.org:8080/#/c/22374/20/java/kudu-hive/build.gradle@37 PS20, Line 37: // Hive Metastore uses hadoop-common-3.10.1 which seems to be having issues finding the correct : // classpath for org.apache.hadoop.fs.BatchListingOperations. This is a workaround to force the : // version which we use for hadoop-common which is said to be compatible with the current Hive : // metastore version according to official Hive documentation: : // https://hive.apache.org/general/downloads/ : : configurations.all { : resolutionStrategy { : force libs.hadoopCommon : } : } > The tests are not happy with that change. Failing with: java.lang.NoClassDefFoundError: org/apache/hadoop/mapred/JobConf Yep, I got the same result. http://gerrit.cloudera.org:8080/#/c/22374/20/java/kudu-hive/build.gradle@37 PS20, Line 37: // Hive Metastore uses hadoop-common-3.10.1 which seems to be having issues finding the correct : // classpath for org.apache.hadoop.fs.BatchListingOperations. This is a workaround to force the : // version which we use for hadoop-common which is said to be compatible with the current Hive : // metastore version according to official Hive documentation: : // https://hive.apache.org/general/downloads/ : : configurations.all { : resolutionStrategy { : force libs.hadoopCommon : } : } > This looks like a cleaner approach. Let me do some reading and get back to > you. It would be great to clarify on this. What I don't like about using resolutionStrategy, is that it looks like a hack, at least at this point. Thanks! -- 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: 20 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: Wed, 19 Mar 2025 04:21:14 +0000 Gerrit-HasComments: Yes
