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

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


The following commit(s) were added to refs/heads/master by this push:
     new 5cf88a5339 [improvement](config) opt the message when missing 
JAVA_HOME for BE (#15045)
5cf88a5339 is described below

commit 5cf88a53392a07450a80d907ce8c9d4bd8582c54
Author: Mingyu Chen <morning...@163.com>
AuthorDate: Wed Dec 14 23:17:46 2022 +0800

    [improvement](config) opt the message when missing JAVA_HOME for BE (#15045)
    
    Make the error message easy to understand
---
 bin/start_be.sh | 8 ++++++++
 conf/be.conf    | 3 +++
 2 files changed, 11 insertions(+)

diff --git a/bin/start_be.sh b/bin/start_be.sh
index b01cd98c7f..f73a6584f3 100755
--- a/bin/start_be.sh
+++ b/bin/start_be.sh
@@ -143,6 +143,14 @@ if [[ -e "${DORIS_HOME}/bin/palo_env.sh" ]]; then
     source "${DORIS_HOME}/bin/palo_env.sh"
 fi
 
+if [[ -z "${JAVA_HOME}" ]]; then
+    echo "The JAVA_HOME environment variable is not defined correctly"
+    echo "This environment variable is needed to run this program"
+    echo "NB: JAVA_HOME should point to a JDK not a JRE"
+    echo "You can set it in be.conf"
+    exit 1
+fi
+
 if [[ ! -d "${LOG_DIR}" ]]; then
     mkdir -p "${LOG_DIR}"
 fi
diff --git a/conf/be.conf b/conf/be.conf
index db67d7b1ff..7dd09d81f3 100644
--- a/conf/be.conf
+++ b/conf/be.conf
@@ -17,6 +17,9 @@
 
 PPROF_TMPDIR="$DORIS_HOME/log/"
 
+# since 1.2, the JAVA_HOME need to be set to run BE process.
+# JAVA_HOME=/path/to/jdk/
+
 # INFO, WARNING, ERROR, FATAL
 sys_log_level = INFO
 


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

Reply via email to