This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch branch-3.5
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.5 by this push:
     new edae8edc3a45 [SPARK-47636][K8S][3.5] Use Java `17` instead of `17-jre` 
image in K8s Dockerfile
edae8edc3a45 is described below

commit edae8edc3a45a860a9402018cb44760266515154
Author: Dongjoon Hyun <dh...@apple.com>
AuthorDate: Thu Mar 28 16:34:25 2024 -0700

    [SPARK-47636][K8S][3.5] Use Java `17` instead of `17-jre` image in K8s 
Dockerfile
    
    ### What changes were proposed in this pull request?
    
    This PR aims to use Java 21 instead of 21-jre in K8s Dockerfile .
    
    ### Why are the changes needed?
    
    Since Apache Spark 3.5.0, SPARK-44153 starts to use `jmap` like the 
following.
    
    
https://github.com/apache/spark/blob/c832e2ac1d04668c77493577662c639785808657/core/src/main/scala/org/apache/spark/util/Utils.scala#L2030
    
    ```
    $ docker run -it --rm eclipse-temurin:17-jre jmap
    /__cacert_entrypoint.sh: line 30: exec: jmap: not found
    ```
    
    ```
    $ docker run -it --rm eclipse-temurin:17 jmap | head -n2
    Usage:
        jmap -clstats <pid>
    ```
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Pass the CIs and manual review.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #45762 from dongjoon-hyun/SPARK-47636.
    
    Authored-by: Dongjoon Hyun <dh...@apple.com>
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
 .../kubernetes/docker/src/main/dockerfiles/spark/Dockerfile             | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/resource-managers/kubernetes/docker/src/main/dockerfiles/spark/Dockerfile 
b/resource-managers/kubernetes/docker/src/main/dockerfiles/spark/Dockerfile
index 88304c87a79c..22d8f1550128 100644
--- a/resource-managers/kubernetes/docker/src/main/dockerfiles/spark/Dockerfile
+++ b/resource-managers/kubernetes/docker/src/main/dockerfiles/spark/Dockerfile
@@ -14,7 +14,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-ARG java_image_tag=17-jre
+ARG java_image_tag=17
 
 FROM eclipse-temurin:${java_image_tag}
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to