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:

(2 comments)

Just took a cursory look.
I have a couple of questions.

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
What happens to build machines where old java version is installed or jenkins 
pre-commit tests launched for this PS?

Does it fail with appropriate error with recommendation action?


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@44
PS1, Line 44:
nit: Remove extra whitespace



--
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, 06 Aug 2025 13:41:12 +0000
Gerrit-HasComments: Yes

Reply via email to