-
Couldn't load subscription status.
- Fork 9.1k
HADOOP-19366. Install OpenJDK 17 in default ubuntu build container #7226
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @pan3793 for the patch, LGTM.
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
@pan3793 Thanks for the contribution! Merged into trunk. @adoroszlai Thanks for the review! |
…pache#7226) Contributed by Cheng Pan. Reviewed-by: Attila Doroszlai <[email protected]> Reviewed-by: Shilun Fan <[email protected]>
…7226) (#7240) Contributed by Cheng Pan. Signed-off-by: Shilun Fan <[email protected]>
Description of PR
Install OpenJDK 17 alongside existing OpenJDK 8 and 11 in the default ubuntu build container. This helps developers who use
start-build-env.shto build and test Hadoop locally with Java 17.Note: the default JAVA_HOME inside the container still points to OpenJDK 8, devs should manually run
export JAVA_HOME=/usr/lib/jvm/java-17-openjdk-arm64to switch to OpenJDK 17How was this patch tested?
Run
./start-build-env.shon macOS M1For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?