Abhishek Chennaka 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
             :   }
             : }
> It seems 'testImplementation libs.hadoopMRClientCommon' is needed for tests
The tests are not happy with that change. Failing with:
java.lang.NoClassDefFoundError: org/apache/hadoop/mapred/JobConf


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
             :   }
             : }
> After looking at this a bit more, I realized that migrating these transitiv
This looks like a cleaner approach. Let me do some reading and get back to you.



--
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 03:39:41 +0000
Gerrit-HasComments: Yes

Reply via email to