chore: Fix jdk documentation for spark#979
chore: Fix jdk documentation for spark#979adi-kmt wants to merge 5 commits intoapache:mainfrom adi-kmt:documentation_jdk_version_fix
Conversation
adi-kmt
changed the title
| ### Supported Apache Spark Versions | ||
| For more details, refer to the documentation: [Apache Spark supported by Comet](overview.md#supported-apache-spark-versions) | ||
|
|
||
| - **Apache Spark 3.3, 3.4, or 3.5** |
There was a problem hiding this comment.
We would like to remove duplications in doc for Supported Apache Versions @adi-kmt Is there any specific reason of having this duplication?
There was a problem hiding this comment.
Main motive was to indicate the different jdk versions wrt to the apache spark versions. Do you think it could be written in a better way?
|
|
||
| - **Apache Spark 3.3, 3.4, or 3.5** | ||
| - **Supported Java Versions**: JDK 8 and above | ||
| - **GLIBC**: 2.17 (CentOS 7) and above |
There was a problem hiding this comment.
Our latest release is built on Ubuntu 20.04 which has glibc 2.31. Could we update that here?
There was a problem hiding this comment.
For pre-built binary release, we can mention what glibc we use. Although I think it doesn't mean older glibc is not supported, users still can build by themselves and use older glibc.
|
@adi-kmt please resolve the conflict |
| @@ -30,17 +30,22 @@ Make sure the following requirements are met and software installed on your mach | |||
|
|
|||
| ### Supported Spark Versions | |||
There was a problem hiding this comment.
| ### Supported Spark Versions |
| - **Scala**: 2.12/2.13 | ||
| - **Apache Spark 4.0** *(Experimental Support - is intended for development/testing use only and should not be used in production yet.)* | ||
| - **Supported Java Versions**: JDK 17/21 | ||
| - **GLIBC**: 2.17 (CentOS 7) and above |
There was a problem hiding this comment.
for GLIBC Im thinking if its good to specify a Linux distribution... I would leave just a version and put a command to check GLIBC like ldd --version
5 participants
Which issue does this PR close?
Closes
Closes #742
Rationale for this change
<-- -->
What changes are included in this PR?
Changed the
installation.mdandHow are these changes tested?
Just documentation change.