chore(explain): Reduce redundant output

Signed-off-by: Alan Tang <[email protected]>

Author: Standing-Man

datafusion/physical-plan/src/projection.rs

@@ -150,38 +150,24 @@ impl DisplayAs for ProjectionExec {
         t: DisplayFormatType,
         f: &mut std::fmt::Formatter,
     ) -> std::fmt::Result {
+        let expr: Vec<String> = self
+            .expr
+            .iter()
+            .map(|(e, alias)| {
+                let e = e.to_string();
+                if &e != alias {
+                    format!("{e} as {alias}")
+                } else {
+                    e
+                }
+            })
+            .collect();
         match t {
             DisplayFormatType::Default | DisplayFormatType::Verbose => {
-                let expr: Vec<String> = self
-                    .expr
-                    .iter()
-                    .map(|(e, alias)| {
-                        let e = e.to_string();
-                        if &e != alias {
-                            format!("{e} as {alias}")
-                        } else {
-                            e
-                        }
-                    })
-                    .collect();
-
                 write!(f, "ProjectionExec: expr=[{}]", expr.join(", "))
             }
             DisplayFormatType::TreeRender => {
-                // TODO: collect info
-                let expr: Vec<String> = self
-                    .expr
-                    .iter()
-                    .map(|(e, alias)| {
-                        let e = e.to_string();
-                        if &e != alias {
-                            format!("{e} as {alias}")
-                        } else {
-                            e
-                        }
-                    })
-                    .collect();
-                write!(f, "ProjectionExec: expr=[{}]", expr.join(", "))
+                write!(f, "expr=[{}]", expr.join(", "))
             }
         }
     }

datafusion/sqllogictest/test_files/explain_tree.slt

@@ -531,7 +531,7 @@ physical_plan
 01)┌───────────────────────────┐
 02)│       ProjectionExec      │
 03)│    --------------------   │
-04)│   ProjectionExec: expr:   │
+04)│           expr:           │
 05)│   [int_col@0 as int_col,  │
 06)│       bigint_col@1 as     │
 07)│      bigint_col, CAST     │
@@ -560,7 +560,7 @@ physical_plan
 01)┌───────────────────────────┐
 02)│       ProjectionExec      │
 03)│    --------------------   │
-04)│   ProjectionExec: expr:   │
+04)│           expr:           │
 05)│   [int_col@0 as int_col,  │
 06)│       bigint_col@1 as     │
 07)│      bigint_col, CAST     │
@@ -589,7 +589,7 @@ physical_plan
 01)┌───────────────────────────┐
 02)│       ProjectionExec      │
 03)│    --------------------   │
-04)│   ProjectionExec: expr:   │
+04)│           expr:           │
 05)│   [int_col@0 as int_col,  │
 06)│       bigint_col@1 as     │
 07)│      bigint_col, CAST     │
@@ -617,7 +617,7 @@ physical_plan
 01)┌───────────────────────────┐
 02)│       ProjectionExec      │
 03)│    --------------------   │
-04)│   ProjectionExec: expr:   │
+04)│           expr:           │
 05)│   [int_col@1 as int_col,  │
 06)│       bigint_col@0 as     │
 07)│     bigint_col, int_col   │
@@ -646,7 +646,7 @@ physical_plan
 01)┌───────────────────────────┐
 02)│       ProjectionExec      │
 03)│    --------------------   │
-04)│   ProjectionExec: expr:   │
+04)│           expr:           │
 05)│   [int_col@0 as int_col,  │
 06)│       bigint_col@1 as     │
 07)│      bigint_col, CAST     │