You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@linkis.apache.org by "zhaowenkai111 (via GitHub)" <gi...@apache.org> on 2023/08/27 14:29:40 UTC

[GitHub] [linkis] zhaowenkai111 opened a new pull request, #4874: Update checkEnv.sh

zhaowenkai111 opened a new pull request, #4874:
URL: https://github.com/apache/linkis/pull/4874

   Added detection script
   pr:
   issue:
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org


Re: [PR] Update checkEnv.sh [linkis]

Posted by "casionone (via GitHub)" <gi...@apache.org>.
casionone closed pull request #4874: Update checkEnv.sh
URL: https://github.com/apache/linkis/pull/4874


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org


[GitHub] [linkis] casionone commented on a diff in pull request #4874: Update checkEnv.sh

Posted by "casionone (via GitHub)" <gi...@apache.org>.
casionone commented on code in PR #4874:
URL: https://github.com/apache/linkis/pull/4874#discussion_r1306686297


##########
linkis-dist/bin/checkEnv.sh:
##########
@@ -18,6 +18,36 @@ shellDir=`dirname $0`
 workDir=`cd ${shellDir}/..;pwd`
 source ${workDir}/bin/common.sh
 source ${workDir}/deploy-config/linkis-env.sh
+source ${workDir}/deploy-config/db.sh
+
+function checkTrino(){
+    TrinoDISAddress="echo $TRINO_DISCOVERY_URL|awk -F'\"' '{print $1}'|awk -F';' '{print $1}'"
+    curl $TrinoDISAddress  > /dev/null 2>&1
+    isSuccess "execute cmd: Trino service check"

Review Comment:
   batter to check server status  is ok 
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org


[GitHub] [linkis] casionone commented on a diff in pull request #4874: Update checkEnv.sh

Posted by "casionone (via GitHub)" <gi...@apache.org>.
casionone commented on code in PR #4874:
URL: https://github.com/apache/linkis/pull/4874#discussion_r1306686195


##########
linkis-dist/bin/checkEnv.sh:
##########
@@ -18,6 +18,36 @@ shellDir=`dirname $0`
 workDir=`cd ${shellDir}/..;pwd`
 source ${workDir}/bin/common.sh
 source ${workDir}/deploy-config/linkis-env.sh
+source ${workDir}/deploy-config/db.sh
+
+function checkTrino(){
+    TrinoDISAddress="echo $TRINO_DISCOVERY_URL|awk -F'\"' '{print $1}'|awk -F';' '{print $1}'"
+    curl $TrinoDISAddress  > /dev/null 2>&1
+    isSuccess "execute cmd: Trino service check"
+}
+
+function checkElasticSearch(){
+    ESRestfulAddress="echo $ES_RESTFUL_URL|awk -F'\"' '{print $1}'|awk -F';' '{print $1}'"
+    curl $ESRestfulAddress> /dev/null 2>&1
+    isSuccess "execute cmd: ElasticSearch service check"
+}
+
+function checkFlink(){
+    FlinkRestfulAddress="echo $Flink_RESTFUL_URL|awk -F'\"' '{print $1}'|awk -F';' '{print $1}'"
+    curl $FlinkRestfulAddress> /dev/null 2>&1
+    isSuccess "execute cmd: Flink service check"
+}
+
+if [ "$ENABLE_TRINO" == "true" ]; then
+  checkTrino
+fi
+
+if [ "$ENABLE_ES" == "true" ]; then
+  checkElasticSearch
+fi
+
+if [ "$ENABLE_FLINK" == "true" ]; then
+fi                                                        

Review Comment:
   nothing todo for "$ENABLE_FLINK" == "true"  ?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org