Zoltan Chovan has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22374 )

Change subject: Upgrade Java dependencies
......................................................................


Patch Set 20:

(1 comment)

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
             :   }
             : }
> > This looks like a cleaner approach. Let me do some reading and get back t
Using platforms, the hierarchy of the project's modules would have to be 
reorganised. The "central" platform project would establish the dependencies 
and the constraints of the transitive dependencies and the dependant projects 
could just include the BOM provided by this platform. Maybe more than one 
platform could be established, e.g. spark/bdr could probably share one, and the 
client another. However this might be a big undertaking, so maybe consider 
using the transitive constraints here locally, to unblock the 1.18 release, and 
follow up with the restructuring in a follow-up patch.



--
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: Mon, 24 Mar 2025 13:42:24 +0000
Gerrit-HasComments: Yes

Reply via email to