puneetdixit200 commented on PR #55985: URL: https://github.com/apache/spark/pull/55985#issuecomment-4528125451
Resolved the two stale review threads after rechecking the current head. The nested `WithCTE` guard now covers both child and subquery traversal paths, and the test no longer includes the redundant explicit normalization call.\n\nVerified locally with Java 17:\n- `git diff --check origin/master HEAD -- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/normalizer/NormalizeCTEIds.scala sql/core/src/test/scala/org/apache/spark/sql/CTEInlineSuite.scala`\n- `build/sbt 'sql/testOnly *CTEInlineSuiteAEOff *CTEInlineSuiteAEOn -- -z SPARK-56921'`\n\nGenerated-by: OpenAI GPT-5 -- 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]
