[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #36847: [SPARK-39448][SQL] Add `ReplaceCTERefWithRepartition` into `nonExcludableRules` list

dongjoon-hyun commented on code in PR #36847: URL: https://github.com/apache/spark/pull/36847#discussion_r895266088 ## sql/core/src/main/scala/org/apache/spark/sql/execution/SparkOptimizer.scala: ## @@ -87,7 +87,8 @@ class SparkOptimizer( GroupBasedRowLevelOperationScanPlan

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #36847: [SPARK-39448][SQL] Add `ReplaceCTERefWithRepartition` into `nonExcludableRules` list

dongjoon-hyun commented on code in PR #36847: URL: https://github.com/apache/spark/pull/36847#discussion_r895266012 ## sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala: ## @@ -4456,6 +4456,20 @@ class SQLQuerySuite extends QueryTest with SharedSparkSession with

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #36847: [SPARK-39448][SQL] Add `ReplaceCTERefWithRepartition` into `nonExcludableRules` list

dongjoon-hyun commented on code in PR #36847: URL: https://github.com/apache/spark/pull/36847#discussion_r895265570 ## sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala: ## @@ -4456,6 +4456,20 @@ class SQLQuerySuite extends QueryTest with SharedSparkSession with