Do not display logical_plan win explain tree mode (#15132)

Author: alamb

datafusion/core/src/physical_planner.rs

@@ -1727,7 +1727,10 @@ impl DefaultPhysicalPlanner {
             let config = &session_state.config_options().explain;
             let explain_format = DisplayFormatType::from_str(&config.format)?;
 
-            if !config.physical_plan_only {
+            let skip_logical_plan = config.physical_plan_only
+                || explain_format == DisplayFormatType::TreeRender;
+
+            if !skip_logical_plan {
                 stringified_plans.clone_from(&e.stringified_plans);
                 if e.logical_optimization_succeeded {
                     stringified_plans.push(e.plan.to_stringified(FinalLogicalPlan));

datafusion/physical-plan/src/display.rs

@@ -34,7 +34,7 @@ use crate::render_tree::RenderTree;
 use super::{accept, ExecutionPlan, ExecutionPlanVisitor};
 
 /// Options for controlling how each [`ExecutionPlan`] should format itself
-#[derive(Debug, Clone, Copy)]
+#[derive(Debug, Clone, Copy, PartialEq)]
 pub enum DisplayFormatType {
     /// Default, compact format. Example: `FilterExec: c12 < 10.0`
     ///