LuciferYang commented on PR #58424: URL: https://github.com/apache/spark/pull/58424#issuecomment-5548572837
rewriteSubqueryPlan runs the full isRepeatablePlan audit before the top-level shape match, so the common `IN (SELECT y FROM t)` (no Join at the top) still pays subqueriesAll and two uncached whole-tree whitelist walks, the second of which runs the per-expression whitelist recursion, and the rule sits in two fixedPoint batches. Move the Project/Join shape match ahead of isRepeatablePlan: semantics are unchanged (both guards still apply) and non-candidate subqueries are left with a handful of pattern matches. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
