update

Author: alamb

datafusion/core/benches/sql_planner.rs

@@ -62,26 +62,29 @@ pub fn create_table_provider(column_prefix: &str, num_columns: usize) -> Arc<Mem
 
 fn create_context() -> SessionContext {
     let ctx = SessionContext::new();
-    ctx.register_table("t1", create_table_provider("a", 200)).unwrap();
-    ctx.register_table("t2", create_table_provider("b", 200)).unwrap();
-    ctx.register_table("t700", create_table_provider("c", 700)).unwrap();
+    ctx.register_table("t1", create_table_provider("a", 200))
+        .unwrap();
+    ctx.register_table("t2", create_table_provider("b", 200))
+        .unwrap();
+    ctx.register_table("t700", create_table_provider("c", 700))
+        .unwrap();
     ctx
 }
 
 fn criterion_benchmark(c: &mut Criterion) {
     let ctx = create_context();
 
     // Test simplest
-    c.bench_function("select_one_from_700", |b| {
-        b.iter(|| {
-            physical_plan(
-                &ctx,
-                "SELECT c1 FROM t700",
-            )
-        })
+    c.bench_function("logical_select_one_from_700", |b| {
+        b.iter(|| logical_plan(&ctx, "SELECT c1 FROM t700"))
+    });
+
+    // Test simplest
+    c.bench_function("physical_select_one_from_700", |b| {
+        b.iter(|| physical_plan(&ctx, "SELECT c1 FROM t700"))
     });
 
-    c.bench_function("trivial join low numbered columns", |b| {
+    c.bench_function("logical_trivial_join_low_numbered_columns", |b| {
         b.iter(|| {
             logical_plan(
                 &ctx,
@@ -91,7 +94,7 @@ fn criterion_benchmark(c: &mut Criterion) {
         })
     });
 
-    c.bench_function("trivial join high numbered columns", |b| {
+    c.bench_function("logical_trivial_join_high_numbered_columns", |b| {
         b.iter(|| {
             logical_plan(
                 &ctx,
@@ -101,7 +104,7 @@ fn criterion_benchmark(c: &mut Criterion) {
         })
     });
 
-    c.bench_function("aggregate with join", |b| {
+    c.bench_function("logical_aggregate_with_join", |b| {
         b.iter(|| {
             logical_plan(
                 &ctx,