HADOOP-19773. [JDK17] Use JDK17 as default in precommit Docker image. #8162

zhtttylz · 2026-01-04T15:35:13Z

Description of PR

HADOOP-19773 [JDK17] Use JDK17 as default in precommit Docker image

The precommit Docker image currently defaults to Java 8, so CI logs for trunk PRs show JDK 8. For example, PR-8151 console output reports:
Default Java | Red Hat, Inc.-1.8.0_472-b08
Update the Java configuration in the Dockerfile and switch the default Java version for the main precommit stage to JDK 17.

For code changes:

No production code changes; CI/build environment only (Dockerfile/Jenkinsfile update).

hadoop-yetus · 2026-01-04T15:39:58Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8162/1/console in case of problems.

hadoop-yetus · 2026-01-04T16:38:56Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8162/2/console in case of problems.

hadoop-yetus · 2026-01-04T16:59:13Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	15m 19s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 0s		codespell was not available.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+0 🆗	shelldocs	0m 0s		Shelldocs was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
			_ trunk Compile Tests _
+0 🆗	mvndep	8m 33s		Maven dependency ordering for branch
+1 💚	shadedclient	26m 40s		branch has no errors when building and testing our client artifacts.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 31s		Maven dependency ordering for patch
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	hadolint	0m 4s		No new issues.
+1 💚	shellcheck	0m 1s		No new issues.
+1 💚	shadedclient	27m 22s		patch has no errors when building and testing our client artifacts.
			_ Other Tests _
+1 💚	asflicense	0m 33s		The patch does not generate ASF License warnings.
		81m 56s

Subsystem	Report/Notes
Docker	ClientAPI=1.52 ServerAPI=1.52 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8162/1/artifact/out/Dockerfile
GITHUB PR	#8162
Optional Tests	dupname asflicense codespell detsecrets hadolint shellcheck shelldocs
uname	Linux 68e2fab80668 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/bin/hadoop.sh
git revision	trunk / `515e06c`
Max. process+thread count	633 (vs. ulimit of 5500)
modules	C: U:
Console output	https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8162/1/console
versions	git=2.25.1 maven=3.9.11 hadolint=1.11.1-0-g0e692dd shellcheck=0.7.0
Powered by	Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

hadoop-yetus · 2026-01-04T17:49:21Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	10m 15s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 0s		codespell was not available.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+0 🆗	shelldocs	0m 1s		Shelldocs was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
			_ trunk Compile Tests _
+0 🆗	mvndep	8m 16s		Maven dependency ordering for branch
+1 💚	shadedclient	25m 45s		branch has no errors when building and testing our client artifacts.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 32s		Maven dependency ordering for patch
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	hadolint	0m 4s		No new issues.
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	shadedclient	24m 41s		patch has no errors when building and testing our client artifacts.
			_ Other Tests _
+1 💚	asflicense	0m 35s		The patch does not generate ASF License warnings.
		72m 51s

Subsystem	Report/Notes
Docker	ClientAPI=1.52 ServerAPI=1.52 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8162/2/artifact/out/Dockerfile
GITHUB PR	#8162
Optional Tests	dupname asflicense codespell detsecrets hadolint shellcheck shelldocs
uname	Linux c81d94970bfb 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/bin/hadoop.sh
git revision	trunk / `8c7f035`
Max. process+thread count	636 (vs. ulimit of 5500)
modules	C: U:
Console output	https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8162/2/console
versions	git=2.25.1 maven=3.9.11 hadolint=1.11.1-0-g0e692dd shellcheck=0.7.0
Powered by	Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

pan3793 · 2026-01-04T18:10:18Z

dev-support/docker/Dockerfile_aarch64

    && apt-get -q install -y --no-install-recommends $(pkg-resolver/resolve.py ubuntu:focal::arch64) \
    && apt-get clean \
-    && update-java-alternatives -s java-1.8.0-openjdk-arm64 \
+    && update-java-alternatives -s java-1.17.0-openjdk-amd64 \


Suggested change

&& update-java-alternatives -s java-1.17.0-openjdk-amd64 \

&& update-java-alternatives -s java-1.17.0-openjdk-arm64 \

Thanks to @pan3793 for reviewing the code; @zhtttylz Thanks for the contribution! make the changes according to Pan’s suggestions.

Thanks @pan3793 for the careful review and the aarch64 pointers! I’ll update Dockerfile_aarch64 to apply your suggestions and push an updated commit.

pan3793 · 2026-01-04T18:10:26Z

dev-support/docker/Dockerfile_aarch64

 ENV PATH="${PATH}:${MAVEN_HOME}/bin"
 # JAVA_HOME must be set in Maven >= 3.5.0 (MNG-6003)
-ENV JAVA_HOME /usr/lib/jvm/java-8-openjdk-arm64
+ENV JAVA_HOME /usr/lib/jvm/java-17-openjdk-amd64


Suggested change

ENV JAVA_HOME /usr/lib/jvm/java-17-openjdk-amd64

ENV JAVA_HOME /usr/lib/jvm/java-17-openjdk-arm64

hadoop-yetus · 2026-01-05T03:40:07Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8162/3/console in case of problems.

hadoop-yetus · 2026-01-05T04:43:14Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	1m 6s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 0s		codespell was not available.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+0 🆗	shelldocs	0m 1s		Shelldocs was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
			_ trunk Compile Tests _
+0 🆗	mvndep	9m 53s		Maven dependency ordering for branch
+1 💚	shadedclient	25m 57s		branch has no errors when building and testing our client artifacts.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 31s		Maven dependency ordering for patch
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	hadolint	0m 2s		No new issues.
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	shadedclient	24m 48s		patch has no errors when building and testing our client artifacts.
			_ Other Tests _
+1 💚	asflicense	0m 34s		The patch does not generate ASF License warnings.
		65m 34s

Subsystem	Report/Notes
Docker	ClientAPI=1.52 ServerAPI=1.52 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8162/3/artifact/out/Dockerfile
GITHUB PR	#8162
Optional Tests	dupname asflicense codespell detsecrets hadolint shellcheck shelldocs
uname	Linux 248d14c67089 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/bin/hadoop.sh
git revision	trunk / `a4db9ed`
Max. process+thread count	634 (vs. ulimit of 5500)
modules	C: U:
Console output	https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8162/3/console
versions	git=2.25.1 maven=3.9.11 hadolint=1.11.1-0-g0e692dd shellcheck=0.7.0
Powered by	Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

cnauroth

+1. Thank you @zhtttylz .

cnauroth · 2026-01-05T17:33:15Z

I committed this to trunk. Thank you @zhtttylz , @pan3793 and @slfan1989 .

cnauroth · 2026-01-05T23:06:21Z

This did not resolve the issue on #8151 . I found there are still some pre-submit steps trying to use old Java 8 platforms for native patches specifically. I filed #8166 to try to resolve it.

github-actions bot added trunk Infra labels Jan 4, 2026

zhtttylz mentioned this pull request Jan 4, 2026

HADOOP-19402. [JDK11] JDiff Support JDK11. #8038

Merged

zhtttylz force-pushed the HADOOP-19773 branch from 515e06c to 8c7f035 Compare January 4, 2026 16:34

pan3793 reviewed Jan 4, 2026

View reviewed changes

zhtttylz changed the title ~~[JDK17] Use JDK17 as default in precommit Docker image.~~ HADOOP-19773. [JDK17] Use JDK17 as default in precommit Docker image. Jan 5, 2026

HADOOP-19773. [JDK17] Use JDK17 as default in precommit Docker image.

a4db9ed

zhtttylz force-pushed the HADOOP-19773 branch from 8c7f035 to a4db9ed Compare January 5, 2026 03:35

pan3793 approved these changes Jan 5, 2026

View reviewed changes

slfan1989 approved these changes Jan 5, 2026

View reviewed changes

cnauroth approved these changes Jan 5, 2026

View reviewed changes

cnauroth closed this in d39a5ec Jan 5, 2026

cnauroth mentioned this pull request Jan 5, 2026

HADOOP-19756 make std_error adopters have less friction with hadoop n… #8151

Closed

4 tasks

	&& update-java-alternatives -s java-1.17.0-openjdk-amd64 \
	&& update-java-alternatives -s java-1.17.0-openjdk-arm64 \

	ENV JAVA_HOME /usr/lib/jvm/java-17-openjdk-amd64
	ENV JAVA_HOME /usr/lib/jvm/java-17-openjdk-arm64

HADOOP-19773. [JDK17] Use JDK17 as default in precommit Docker image. #8162

HADOOP-19773. [JDK17] Use JDK17 as default in precommit Docker image. #8162

Conversation

zhtttylz commented Jan 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description of PR

For code changes:

Uh oh!

hadoop-yetus commented Jan 4, 2026

Uh oh!

hadoop-yetus commented Jan 4, 2026

Uh oh!

hadoop-yetus commented Jan 4, 2026

Uh oh!

hadoop-yetus commented Jan 4, 2026

Uh oh!

pan3793 Jan 4, 2026

Choose a reason for hiding this comment

Uh oh!

slfan1989 Jan 5, 2026

Choose a reason for hiding this comment

Uh oh!

zhtttylz Jan 5, 2026

Choose a reason for hiding this comment

Uh oh!

pan3793 Jan 4, 2026

Choose a reason for hiding this comment

Uh oh!

hadoop-yetus commented Jan 5, 2026

Uh oh!

hadoop-yetus commented Jan 5, 2026

Uh oh!

cnauroth left a comment

Choose a reason for hiding this comment

Uh oh!

cnauroth commented Jan 5, 2026

Uh oh!

cnauroth commented Jan 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

zhtttylz commented Jan 4, 2026 •

edited

Loading