bug fix

apache · Apr 28, 2023 · 68f038f · 68f038f
1 parent ad145a5
commit 68f038f
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/benchmarks/bench.sh b/benchmarks/bench.sh
@@ -32,6 +32,8 @@ COMMAND=
 BENCHMARK=all
 DATAFUSION_DIR=${DATAFUSION_DIR:-$SCRIPT_DIR/..}
 DATA_DIR=${DATA_DIR:-$SCRIPT_DIR/data}
+#CARGO_COMMAND=$CARGO_COMMAND:"cargo run --release"}
+CARGO_COMMAND=${CARGO_COMMAND:-"cargo run --profile release-nonlto"}  # TEMP: for faster iterations
 
 usage() {
     echo "
@@ -151,8 +153,6 @@ main() {
             BRANCH_NAME=$(cd ${DATAFUSION_DIR} && git rev-parse --abbrev-ref HEAD)
             BRANCH_NAME=${BRANCH_NAME//\//_} # mind blowing syntax to replace / with _
             RESULTS_DIR=${RESULTS_DIR:-"$SCRIPT_DIR/results/$BRANCH_NAME"}
-            #CARGO_COMMAND=$CARGO_COMMAND:"cargo run --release"}
-            CARGO_COMMAND=${CARGO_COMMAND:-"cargo run --profile release-nonlto"}  # TEMP: for faster iterations
 
             echo "***************************"
             echo "DataFusion Benchmark Script"
@@ -184,7 +184,7 @@ main() {
                     usage
                     ;;
             esac
-            popd "${DATAFUSION_DIR}/benchmarks" > /dev/null
+            popd > /dev/null
             echo "Done"
             ;;
         compare)
@@ -234,7 +234,9 @@ data_tpch() {
         echo " parquet files exist ($FILE exists)."
     else
         echo " creating parquet files using benchmark binary ..."
+        pushd "${SCRIPT_DIR}" > /dev/null
         $CARGO_COMMAND --bin tpch -- convert --input "${DATA_DIR}" --output "${DATA_DIR}" --format parquet
+        popd > /dev/null
     fi
 }