Ashwani Raina has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23259 )

Change subject: [build][java] KUDU-3681: Move to JDK17
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/23259/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/23259/1//COMMIT_MSG@7
PS1, Line 7: Move to JDK17
> our precommit testing environment will need upgrading to facilitate this ch
If we upgrade test environment without this change, would builds pass? In other 
words, does upgraded version have backward compatibility?


http://gerrit.cloudera.org:8080/#/c/23259/1//COMMIT_MSG@12
PS1, Line 12: javaCompatibility to 17 in gradle.properties
Do you think System requirements also need to change in java/README.adoc?


http://gerrit.cloudera.org:8080/#/c/23259/1/java/gradle/dependencies.gradle
File java/gradle/dependencies.gradle:

http://gerrit.cloudera.org:8080/#/c/23259/1/java/gradle/dependencies.gradle@111
PS1, Line 111: hadoopHdfs
Is this picked from shadow jar or native system?



--
To view, visit http://gerrit.cloudera.org:8080/23259
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1f73c0d3a79e1c37b12a173881273b4b68d718b3
Gerrit-Change-Number: 23259
Gerrit-PatchSet: 1
Gerrit-Owner: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
Gerrit-Comment-Date: Wed, 03 Sep 2025 13:24:12 +0000
Gerrit-HasComments: Yes

Reply via email to