fix issue

Signed-off-by: YanxuanLiu <[email protected]>

Author: nvauto

.github/workflows/shell-check.yml

@@ -32,8 +32,10 @@ jobs:
         uses: NVIDIA/spark-rapids-common/shell-check@main
         with:
           excluded_codes:
-            SC2181
+            SC2181,
+            SC1090
 
           # code explanation:
           # SC2181: note: Check exit code directly with e.g. 'if mycmd;', not indirectly with $?.
+          # SC1090: warning: ShellCheck can't follow non-constant source. Use a directive to specify location. [SC1090]
 

python/benchmark/databricks/benchmark_utils.sh

@@ -82,7 +82,7 @@ run_bm() {
 
     algorithm=$1
 
-    params_delimited=$( echo $@ | sed -e 's/^/"/g' | sed -e 's/$/"/g' | sed -e 's/ /", "/g' )
+    params_delimited=$( echo "$@" | sed -e 's/^/"/g' | sed -e 's/$/"/g' | sed -e 's/ /", "/g' )
 
 json_string=`cat <<EOF
 {
@@ -132,7 +132,7 @@ EOF
         echo -n "."
         if [[ $TIME_LIMIT != "" ]] && (( duration > TIME_LIMIT ))
         then
-            echo "\ntime limit of $TIME_LIMIT minutes exceeded, canceling run"
+            printf "\ntime limit of $TIME_LIMIT minutes exceeded, canceling run\n"
             databricks jobs cancel-run $run_id --profile $DB_PROFILE
         fi
         sleep 10