[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r453548369 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,24 @@ object

[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r453548369 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,24 @@ object

[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452992807 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala ## @@ -161,7 +161,8 @@ abstract class

[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452913572 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelationSuite.scala ## @@ -67,6 +68,28 @@ class

[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452848538 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,25 @@ object

[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452842236 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,25 @@ object

[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452703468 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,25 @@ object

[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452691649 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,25 @@ object

[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452690143 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,25 @@ object

[GitHub] [spark] peter-toth commented on a change in pull request #29053: [SPARK-32241][SQL] Remove empty children of union

peter-toth commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452669317 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,25 @@ object