Stop pretty printing in benchmark when there are no results (apache#3974

)
Dandandan · Oct 27, 2022 · 6860ae4 · 6860ae4
1 parent 4b5d720
commit 6860ae4
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/benchmarks/src/bin/tpch.rs b/benchmarks/src/bin/tpch.rs
@@ -326,7 +326,11 @@ async fn execute_query(
             "=== Physical plan with metrics ===\n{}\n",
             DisplayableExecutionPlan::with_metrics(physical_plan.as_ref()).indent()
         );
-        pretty::print_batches(&result)?;
+        if !result.is_empty() {
+            // do not call print_batches if there are no batches as the result is confusing
+            // and makes it look like there is a batch with no columns
+            pretty::print_batches(&result)?;
+        }
     }
     Ok(result)
 }