chore: fix stackoverflow in CleanupUnusedCTEOptimizer (#18476)

SkyFan2002 · web-flow · commit ca842e5f674b · 2025-08-04T19:10:08.000+08:00
diff --git a/src/query/sql/src/planner/optimizer/optimizers/operator/cte/cleanup_unused_cte.rs b/src/query/sql/src/planner/optimizer/optimizers/operator/cte/cleanup_unused_cte.rs
@@ -56,6 +56,7 @@ impl CleanupUnusedCTEOptimizer {
     }
 
     /// Remove unused CTEs from the expression tree and update ref_count
+    #[recursive::recursive]
     fn remove_unused_ctes(
         s_expr: &SExpr,
         referenced_ctes: &HashMap<String, usize>,

Original file line number	Diff line number	Diff line change
`@@ -56,6 +56,7 @@ impl CleanupUnusedCTEOptimizer {`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`/// Remove unused CTEs from the expression tree and update ref_count`
	`59`	`+ #[recursive::recursive]`
`59`	`60`	`fn remove_unused_ctes(`
`60`	`61`	`s_expr: &SExpr,`
`61`	`62`	`referenced_ctes: &HashMap<String, usize>,`