You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by bi...@apache.org on 2017/02/10 09:03:38 UTC

[04/17] kylin git commit: minor, clean up kylin.sh a little

minor, clean up kylin.sh a little


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/df3ecd32
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/df3ecd32
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/df3ecd32

Branch: refs/heads/KYLIN-2428
Commit: df3ecd324a6442fee52f146d930cc5f84dafd945
Parents: d135bdb
Author: Yang Li <li...@apache.org>
Authored: Wed Feb 8 20:42:02 2017 +0800
Committer: Yang Li <li...@apache.org>
Committed: Wed Feb 8 20:42:02 2017 +0800

----------------------------------------------------------------------
 build/bin/kylin.sh | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/df3ecd32/build/bin/kylin.sh
----------------------------------------------------------------------
diff --git a/build/bin/kylin.sh b/build/bin/kylin.sh
index 7813b79..cc571e5 100644
--- a/build/bin/kylin.sh
+++ b/build/bin/kylin.sh
@@ -60,8 +60,7 @@ then
         PID=`cat $KYLIN_HOME/pid`
         if ps -p $PID > /dev/null
         then
-          echo "Kylin is running, stop it first"
-          exit 1
+          quit "Kylin is running, stop it first"
         fi
     fi
     
@@ -81,7 +80,7 @@ then
     spring_profile=`bash ${dir}/get-properties.sh kylin.security.profile`
     if [ -z "$spring_profile" ]
     then
-        quit 'please set kylin.security.profile in kylin.properties, options are: testing, ldap, saml.'
+        quit 'Please set kylin.security.profile in kylin.properties, options are: testing, ldap, saml.'
     else
         verbose "kylin.security.profile is set to $spring_profile"
     fi
@@ -95,10 +94,7 @@ then
     kylin_rest_address_arr=(${kylin_rest_address//;/ })
     nc -z -w 5 ${kylin_rest_address_arr[0]} ${kylin_rest_address_arr[1]} 1>/dev/null 2>&1; nc_result=$?
     if [ $nc_result -eq 0 ]; then
-        echo "port ${kylin_rest_address} is not available, could not start Kylin"
-        exit 1
-    else
-        echo "port ${kylin_rest_address} is available"
+        quit "Port ${kylin_rest_address} is not available, could not start Kylin."
     fi
 
     #debug if encounter NoClassDefError
@@ -155,7 +151,7 @@ then
 
 elif [ "$1" = "diag" ]
 then
-    echo "kylin.sh diag no longer supported, use diag.sh instead"
+    echo "'kylin.sh diag' no longer supported, use diag.sh instead"
     exit 0
 
 # tool command