[systemds] branch main updated: [MINOR] Fix bug in runscript

[markd]

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

markd pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/systemds.git


The following commit(s) were added to refs/heads/main by this push:
     new f357c776be [MINOR] Fix bug in runscript
f357c776be is described below

commit f357c776beb85127bfdb3ddd51b37a00f9263f10
Author: Mark Dokter <m...@dokter.cc>
AuthorDate: Mon May 16 11:28:26 2022 +0200

    [MINOR] Fix bug in runscript
    
    This commit fixes a bug that was introduced in commit 
f33b516d102115433ad101d0f76136cab92d01ae where an if should have been an elif.
    Additionally, there are some minor cleanups to silence some linter warnings 
(IntelliJ)
---
 bin/systemds | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/bin/systemds b/bin/systemds
index ffff838991..b88da6e00d 100755
--- a/bin/systemds
+++ b/bin/systemds
@@ -82,11 +82,8 @@ ordered_find() {
 if [ -n "$SYSTEMDS_STANDALONE_OPTS" ]; then
        print_out "Overriding SYSTEMDS_STANDALONE_OPTS with env var: 
$SYSTEMDS_STANDALONE_OPTS"
 else
-  # specify paramteters  to java when running locally here
-  SYSTEMDS_STANDALONE_OPTS="\
-      -Xmx4g\
-      -Xms4g\
-      -Xmn400m "
+  # specify parameters  to java when running locally here
+  SYSTEMDS_STANDALONE_OPTS="-Xmx4g -Xms4g -Xmn400m "
 fi
 
 if [ -n "$SYSTEMDS_REMOTE_DEBUGGING" ]; then
@@ -363,7 +360,7 @@ fi
 read -r -d '' -a myArray < <( echo "$@" )
 INDEX=0
 for i in "${myArray[@]}"; do
-  if [[ ${myArray[INDEX]} == *-exec* ]]; then
+  if [[ "$i" == *-exec* ]]; then
     SYSDS_EXEC_MODE="${myArray[((INDEX+1))]}"
     break;
   fi
@@ -414,7 +411,7 @@ CLASSPATH=$(echo "${CLASSPATH}" | tr -d '[:space:]')
 if [ $PRINT_SYSDS_HELP == 1 ]; then
   echo "----------------------------------------------------------------------"
   echo "Further help on SystemDS arguments:"
-  java -cp $CLASSPATH org.apache.sysds.api.DMLScript -help
+  java -cp "$CLASSPATH" org.apache.sysds.api.DMLScript -help
   exit 1
 fi
 
@@ -443,7 +440,7 @@ if [ $WORKER == 1 ]; then
   print_out "Executing command: $CMD"
   print_out  ""
 
-if [ $FEDMONITORING == 1 ]; then
+elif [ "$FEDMONITORING" == 1 ]; then
   print_out "#"
   print_out "#  starting Federated backend monitoring on port $PORT"
   print_out 
"###############################################################################"