s-maling-celonis commented on PR #58856: URL: https://github.com/apache/spark/pull/58856#issuecomment-5886996815
Thanks @cloud-fan for the follow-up review. Both comments are addressed in 8295f7e5b8d (rebased onto the latest master): - CTE definitions are now presented to the hint rules as one `WithCTE` tree and converted back afterwards, so each rule receives one root per iteration regardless of the number of CTE definitions; subqueries keep their separate re-entry - a `HybridAnalyzerSuite` test runs `fromLegacyAnalyzer` with distinct legacy and single-pass hint rules and checks that each mode runs only its own sequence The PR description is updated to match. -- 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]
