Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-13 Thread via GitHub
cloud-fan closed pull request #50546: [SPARK-51752][SQL] Enable rCTE referencing from within a CTE URL: https://github.com/apache/spark/pull/50546 -- 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

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-13 Thread via GitHub
cloud-fan commented on PR #50546: URL: https://github.com/apache/spark/pull/50546#issuecomment-2799941944 thanks, merging to master! -- 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 c

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-13 Thread via GitHub
Pajaraja commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2041085999 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CTESubstitution.scala: ## @@ -358,7 +385,7 @@ object CTESubstitution extends Rule[LogicalPlan] {

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-12 Thread via GitHub
Pajaraja commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2040788341 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CTESubstitution.scala: ## @@ -255,6 +273,21 @@ object CTESubstitution extends Rule[LogicalPlan] {

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-12 Thread via GitHub
cloud-fan commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2040663825 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CTESubstitution.scala: ## @@ -220,18 +224,31 @@ object CTESubstitution extends Rule[LogicalPlan]

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-12 Thread via GitHub
cloud-fan commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2040664694 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CTESubstitution.scala: ## @@ -358,7 +385,7 @@ object CTESubstitution extends Rule[LogicalPlan] {

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-12 Thread via GitHub
cloud-fan commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2040664563 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CTESubstitution.scala: ## @@ -255,6 +273,21 @@ object CTESubstitution extends Rule[LogicalPlan]

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-12 Thread via GitHub
cloud-fan commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2040663762 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CTESubstitution.scala: ## @@ -220,18 +224,31 @@ object CTESubstitution extends Rule[LogicalPlan]

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-11 Thread via GitHub
Pajaraja commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2039960513 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveWithCTE.scala: ## @@ -83,8 +83,21 @@ object ResolveWithCTE extends Rule[LogicalPlan] {

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-11 Thread via GitHub
Pajaraja commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2039667154 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CTESubstitution.scala: ## @@ -220,18 +224,31 @@ object CTESubstitution extends Rule[LogicalPlan]

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-11 Thread via GitHub
cloud-fan commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2039400525 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveWithCTE.scala: ## @@ -83,8 +83,21 @@ object ResolveWithCTE extends Rule[LogicalPlan] {

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-11 Thread via GitHub
cloud-fan commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2039399624 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveWithCTE.scala: ## @@ -83,8 +83,21 @@ object ResolveWithCTE extends Rule[LogicalPlan] {

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-11 Thread via GitHub
cloud-fan commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2039396283 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CTESubstitution.scala: ## @@ -220,18 +224,31 @@ object CTESubstitution extends Rule[LogicalPlan]

Re: [PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-11 Thread via GitHub
cloud-fan commented on code in PR #50546: URL: https://github.com/apache/spark/pull/50546#discussion_r2039394412 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CTESubstitution.scala: ## @@ -202,14 +202,18 @@ object CTESubstitution extends Rule[LogicalPlan]

[PR] [SPARK-51752][SQL] Enable rCTE referencing from within a CTE [spark]

2025-04-09 Thread via GitHub
Pajaraja opened a new pull request, #50546: URL: https://github.com/apache/spark/pull/50546 ### What changes were proposed in this pull request? Enabling the possibility of a CTE referencing the recursive CTE it is inside of. This is done by modifying the CTESubstitution file, consist