-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-44112][BUILD][INFRA][DOCS] Drop support for Java 8 and Java 11 #43005
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
Changes from all commits
4079d7a
4c11396
f7ae713
cb5f8de
72b51ad
4c6f254
50d7693
7ce64d8
bbeba9a
278cac6
3f11512
bbbed13
c0a330d
6ca32e3
2ea3160
54f40ab
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -30,7 +30,7 @@ RUN apt-get update && apt-get install -y \ | |
| pkg-config \ | ||
| curl \ | ||
| wget \ | ||
| openjdk-8-jdk \ | ||
| openjdk-17-jdk-headless \ | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @dongjoon-hyun @HyukjinKwon should we change this?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. cc @Yikun
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @dongjoon-hyun So in this PR, should we revert this change and consider it as a followup?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Emm, jdk install in here seems useless, because we are using github action to install the java. But if CI passed, it is ok.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks @Yikun ~ Let me continue to monitor the running status of GA. |
||
| gfortran \ | ||
| libopenblas-dev \ | ||
| liblapack-dev \ | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -27,7 +27,7 @@ license: | | |
| ## Apache Maven | ||
|
|
||
| The Maven-based build is the build of reference for Apache Spark. | ||
| Building Spark using Maven requires Maven 3.9.4 and Java 8/11/17. | ||
| Building Spark using Maven requires Maven 3.9.4 and Java 17. | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we add |
||
| Spark requires Scala 2.13; support for Scala 2.12 was removed in Spark 4.0.0. | ||
|
|
||
| ### Setting up Maven's Memory Usage | ||
|
|
||
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.
It seems easier to install Java this way, the daily test of branch-3.x all with
java: 8