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

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


Patch Set 22: Code-Review+1

(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: shadowJar {
             :   dependencies {
             :     exclude(dependency("log4j::.*"))
             :     exclude(dependency("org.apache.hadoop::.*"))
             :     exclude(dependency("org.apache.hive::.*"))
             :     exclude(dependency("org.apache.hbase::.*"))
             :     exclude(dependency("org.apache.hbase.thirdparty::.*"))
             :     exclude(dependency("junit::.*"))
             :     exclude(dependency("javax.servlet.jsp::.*"))
             :     exclude(dependency("javax.jdo::.*"))
             :
> Q. Why is the test failing with hadoop-common:3.4.1 and not with 
> hadoop-common:3.3.1?
While both the versions have DistributedFileSystems class extending 
BatchListingOperations, the FileSystems class from hadoop-common:3.3.1  was 
present from spark-core_2.12:3.2.4 -> hadoop-client-api:3.3.1 as a transitive 
dependency.

So, why the test was failing with 3.4.1, but not with 3.3.1 version of the 
hadoop-common package?  The cited snippet above only explains from where a 
particular version came as a transitive dependency, but I could not find the 
explanation behind the failure of the test.


http://gerrit.cloudera.org:8080/#/c/22374/22/java/kudu-test-utils/build.gradle
File java/kudu-test-utils/build.gradle:

PS22:
If this file hasn't actually changed between the base and PS22, maybe revert 
the formatting-only updates to keep this changelist cleaner?



--
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: 22
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, 02 Apr 2025 01:16:26 +0000
Gerrit-HasComments: Yes

Reply via email to