chore: remove explain join estimated_rows (#13212)

* remove explain join estimated_rows * sqllogictest
datafuselabs · Oct 11, 2023 · 4567c4c · 4567c4c · vercel · Oct 11, 2023
1 parent be202c9
commit 4567c4c
Show file tree

Hide file tree

Showing 4 changed files with 73 additions and 74 deletions.
diff --git a/src/query/sql/src/executor/format.rs b/src/query/sql/src/executor/format.rs
@@ -97,14 +97,14 @@ impl PhysicalPlan {
                     FormatTreeNode::with_children("Probe".to_string(), vec![probe_child]),
                 ];
 
-                let estimated_rows = if let Some(info) = &plan.stat_info {
+                let _estimated_rows = if let Some(info) = &plan.stat_info {
                     format!("{0:.2}", info.estimated_rows)
                 } else {
                     String::from("None")
                 };
 
                 Ok(FormatTreeNode::with_children(
-                    format!("HashJoin: {} (rows: {})", plan.join_type, estimated_rows),
+                    format!("HashJoin: {}", plan.join_type),
                     children,
                 ))
             }
@@ -117,14 +117,14 @@ impl PhysicalPlan {
                     FormatTreeNode::with_children("Right".to_string(), vec![right_child]),
                 ];
 
-                let estimated_rows = if let Some(info) = &plan.stat_info {
+                let _estimated_rows = if let Some(info) = &plan.stat_info {
                     format!("{0:.2}", info.estimated_rows)
                 } else {
                     String::from("none")
                 };
 
                 Ok(FormatTreeNode::with_children(
-                    format!("RangeJoin: {} (rows: {})", plan.join_type, estimated_rows),
+                    format!("RangeJoin: {}", plan.join_type,),
                     children,
                 ))
             }

diff --git a/tests/sqllogictests/suites/mode/standalone/explain/join_reorder/chain.test b/tests/sqllogictests/suites/mode/standalone/explain/join_reorder/chain.test
@@ -502,7 +502,7 @@ HashJoin
 query T
 explain join select * from t right anti join t1 on t1.a = t.a
 ----
-HashJoin: LEFT ANTI (rows: 10.00)
+HashJoin: LEFT ANTI
 ├── Build
 │   └── Scan: default.join_reorder.t (read rows: 1)
 └── Probe

diff --git a/tests/sqllogictests/suites/mode/standalone/explain_native/join_reorder/chain.test b/tests/sqllogictests/suites/mode/standalone/explain_native/join_reorder/chain.test
@@ -502,7 +502,7 @@ HashJoin
 query T
 explain join select * from t right anti join t1 on t1.a = t.a
 ----
-HashJoin: LEFT ANTI (rows: 10.00)
+HashJoin: LEFT ANTI
 ├── Build
 │   └── Scan: default.join_reorder.t (read rows: 1)
 └── Probe