From 0f315783e25db1a5e7381043f4ca01f71239c5f0 Mon Sep 17 00:00:00 2001 From: Dennis Felsing Date: Thu, 13 Feb 2025 16:45:40 +0000 Subject: [PATCH] explain: Switch default to global plan instead of physical plan Was forgotten in https://github.com/MaterializeInc/materialize/pull/31185 > 3c1c409 changes the default for a bare EXPLAIN to use EXPLAIN PHYSICAL PLAN. But I only changed the default in order to catch every bare EXPLAIN, and I change the default back in c97eb41. --- src/sql-parser/src/ast/defs/statement.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/sql-parser/src/ast/defs/statement.rs b/src/sql-parser/src/ast/defs/statement.rs index b5c61bee1c8ea..e9f693a2bc0a2 100644 --- a/src/sql-parser/src/ast/defs/statement.rs +++ b/src/sql-parser/src/ast/defs/statement.rs @@ -3837,7 +3837,7 @@ pub struct ExplainPlanStatement { impl ExplainPlanStatement { pub fn stage(&self) -> ExplainStage { - self.stage.unwrap_or(ExplainStage::PhysicalPlan) + self.stage.unwrap_or(ExplainStage::GlobalPlan) } pub fn format(&self) -> ExplainFormat {