[incubator-uniffle] branch master updated: [#674] feat(docker): use JDK11 as the default java version in Dockerfile (#683)

[zuston]

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

zuston pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
     new 31e4a3f2 [#674] feat(docker): use JDK11 as the default java version in 
Dockerfile (#683)
31e4a3f2 is described below

commit 31e4a3f2f03ed1a71c85d575c098e394ebece3db
Author: Junfan Zhang <zus...@apache.org>
AuthorDate: Thu Mar 9 15:35:21 2023 +0800

    [#674] feat(docker): use JDK11 as the default java version in Dockerfile 
(#683)
    
    ### What changes were proposed in this pull request?
    
    Use JDK11 as the default java version in Dockerfile when deploying shuffle 
servers on K8s.
    
    ### Why are the changes needed?
    
    As the GC problems mentioned in #674, I upgraded the JVM version from JDK8 
to JDK11 for shuffle server. And after benchmark, I found the effect of the 
upgrade is remarkable. And it also works well with Hadoop3.1.1
    
    Based on above practice, it's better to use JDK11 as the default java 
version to improve stability
    
    ### Does this PR introduce _any_ user-facing change?
    
    Yes
    
    ### How was this patch tested?
    
    Have tested in production env.
    
    Co-authored-by: zhangjunfan <zhangjun...@qiyi.com>
---
 deploy/kubernetes/docker/base/centos/Dockerfile | 2 +-
 deploy/kubernetes/docker/base/debian/Dockerfile | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/deploy/kubernetes/docker/base/centos/Dockerfile 
b/deploy/kubernetes/docker/base/centos/Dockerfile
index 5a62e64e..8aa7695e 100644
--- a/deploy/kubernetes/docker/base/centos/Dockerfile
+++ b/deploy/kubernetes/docker/base/centos/Dockerfile
@@ -16,7 +16,7 @@
 #
 
 # this docker file serves the rss server base image for CentOS/RedHat 
distributions
-ARG BASE_IMAGE=eclipse-temurin:8-jdk-centos7
+ARG BASE_IMAGE=eclipse-temurin:11-jdk-centos7
 FROM ${BASE_IMAGE}
 
 RUN yum install lzo zlib zlib-devel lzop lsof netcat dnsutils less procps 
iputils-ping -y && \
diff --git a/deploy/kubernetes/docker/base/debian/Dockerfile 
b/deploy/kubernetes/docker/base/debian/Dockerfile
index 6c639f91..901d0fd1 100644
--- a/deploy/kubernetes/docker/base/debian/Dockerfile
+++ b/deploy/kubernetes/docker/base/debian/Dockerfile
@@ -16,7 +16,7 @@
 #
 
 # this docker file serves the rss server base image for Debian/Ubuntu releated 
distributions
-ARG BASE_IMAGE=eclipse-temurin:8-jdk
+ARG BASE_IMAGE=eclipse-temurin:11-jdk
 FROM ${BASE_IMAGE}
 
 RUN apt-get update && \