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

joemcdonnell pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git

commit bfa4402c131d0815088f43445478fcfe0e1f185a
Author: Riza Suminto <[email protected]>
AuthorDate: Fri Apr 11 19:10:29 2025 -0700

    IMPALA-13956: Move java version check after yum install
    
    IMPALA-13920 adds java and javac version check in bootstrap_system.sh
    This patch move that check after 'sudo yum install' command so that
    Impala build in redhat machine can work.
    
    Testing:
    Build Impala using jenkins and Redhat 8.6 machine.
    
    Change-Id: I25b26c146bf13138741272cd73727e7244462249
    Reviewed-on: http://gerrit.cloudera.org:8080/22772
    Reviewed-by: Csaba Ringhofer <[email protected]>
    Tested-by: Impala Public Jenkins <[email protected]>
---
 bin/bootstrap_system.sh | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/bin/bootstrap_system.sh b/bin/bootstrap_system.sh
index ed5d44564..1bf77a9d2 100755
--- a/bin/bootstrap_system.sh
+++ b/bin/bootstrap_system.sh
@@ -278,12 +278,6 @@ ubuntu sudo update-java-alternatives -l || true
 ubuntu sudo update-java-alternatives -v -s \
     java-1.${UBUNTU_JAVA_VERSION}.0-openjdk-${UBUNTU_PACKAGE_ARCH}
 
-# update-java-alternatives may not take effect if there is a Java in PATH
-which java
-java -version
-which javac
-javac -version
-
 redhat sudo yum install -y file gawk gcc gcc-c++ git krb5-devel krb5-server \
         krb5-workstation libevent-devel libffi-devel make openssl-devel 
cyrus-sasl \
         cyrus-sasl-gssapi cyrus-sasl-devel cyrus-sasl-plain \
@@ -293,6 +287,12 @@ redhat sudo yum install -y file gawk gcc gcc-c++ git 
krb5-devel krb5-server \
         net-tools langpacks-en glibc-langpack-en libxml2-devel libxslt-devel \
         java-${REDHAT_JAVA_VERSION}-openjdk-src 
java-${REDHAT_JAVA_VERSION}-openjdk-devel
 
+# update-java-alternatives may not take effect if there is a Java in PATH
+which java
+java -version
+which javac
+javac -version
+
 # fuse-devel doesn't exist for Redhat 9
 redhat7 sudo yum install -y fuse-devel curl
 redhat8 sudo yum install -y fuse-devel curl

Reply via email to