[GitHub] [spark] wangyum commented on a change in pull request #35214: [SPARK-37915][SQL] Combine unions if there is a project between them

2022-01-25 Thread GitBox
wangyum commented on a change in pull request #35214: URL: https://github.com/apache/spark/pull/35214#discussion_r791357478 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -1300,16 +1300,22 @@ object InferFiltersFromConst

[GitHub] [spark] wangyum commented on a change in pull request #35214: [SPARK-37915][SQL] Combine unions if there is a project between them

2022-01-24 Thread GitBox
wangyum commented on a change in pull request #35214: URL: https://github.com/apache/spark/pull/35214#discussion_r791357478 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -1300,16 +1300,22 @@ object InferFiltersFromConst

[GitHub] [spark] wangyum commented on a change in pull request #35214: [SPARK-37915][SQL] Combine unions if there is a project between them

2022-01-21 Thread GitBox
wangyum commented on a change in pull request #35214: URL: https://github.com/apache/spark/pull/35214#discussion_r790101055 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -1322,6 +1322,25 @@ object CombineUnions extends

[GitHub] [spark] wangyum commented on a change in pull request #35214: [SPARK-37915][SQL] Combine unions if there is a project between them

2022-01-19 Thread GitBox
wangyum commented on a change in pull request #35214: URL: https://github.com/apache/spark/pull/35214#discussion_r787589420 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -1322,6 +1322,12 @@ object CombineUnions extends

[GitHub] [spark] wangyum commented on a change in pull request #35214: [SPARK-37915][SQL] Combine unions if there is a project between them

2022-01-19 Thread GitBox
wangyum commented on a change in pull request #35214: URL: https://github.com/apache/spark/pull/35214#discussion_r787589160 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -1322,6 +1322,12 @@ object CombineUnions extends

[GitHub] [spark] wangyum commented on a change in pull request #35214: [SPARK-37915][SQL] Combine unions if there is a project between them

2022-01-17 Thread GitBox
wangyum commented on a change in pull request #35214: URL: https://github.com/apache/spark/pull/35214#discussion_r786435360 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -1322,6 +1322,12 @@ object CombineUnions extends

[GitHub] [spark] wangyum commented on a change in pull request #35214: [SPARK-37915][SQL] Combine unions if there is a project between them

2022-01-17 Thread GitBox
wangyum commented on a change in pull request #35214: URL: https://github.com/apache/spark/pull/35214#discussion_r786435360 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -1322,6 +1322,12 @@ object CombineUnions extends

[GitHub] [spark] wangyum commented on a change in pull request #35214: [SPARK-37915][SQL] Combine unions if there is a project between them

2022-01-17 Thread GitBox
wangyum commented on a change in pull request #35214: URL: https://github.com/apache/spark/pull/35214#discussion_r786418127 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -1322,6 +1322,12 @@ object CombineUnions extends

[GitHub] [spark] wangyum commented on a change in pull request #35214: [SPARK-37915][SQL] Combine unions if there is a project between them

2022-01-17 Thread GitBox
wangyum commented on a change in pull request #35214: URL: https://github.com/apache/spark/pull/35214#discussion_r786417364 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -762,22 +762,22 @@ object PushProjectionThroughUn