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

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


Patch Set 23:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22374/23/java/gradle/shadow.gradle
File java/gradle/shadow.gradle:

http://gerrit.cloudera.org:8080/#/c/22374/23/java/gradle/shadow.gradle@51
PS23, Line 51:   // Excluding classes with Java21 as Gradle 7 doesn't recognize 
them and
             :   // Java9 version specific classes(module-info.class) as we 
compile with
             :   // Java8 and don't need them anyway.
> I guess the current list of versions to exclude was found by trial-and-erro
As discussed offline,
1. We are excluding any unsupported Java versions by Gradle 7.6
2&3. Adding a TODO on this but going with the current approach of excluding all 
MR-JAR specific files.



--
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: 23
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: Fri, 04 Apr 2025 00:12:31 +0000
Gerrit-HasComments: Yes

Reply via email to