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

2020-07-10 Thread GitBox
cloud-fan commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452850859 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelationSuite.scala ## @@ -67,6 +68,28 @@ class

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

2020-07-10 Thread GitBox
cloud-fan commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452695403 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,25 @@ object

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

2020-07-10 Thread GitBox
cloud-fan commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452670472 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,25 @@ object

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

2020-07-10 Thread GitBox
cloud-fan commented on a change in pull request #29053: URL: https://github.com/apache/spark/pull/29053#discussion_r452656336 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/PropagateEmptyRelation.scala ## @@ -50,8 +50,25 @@ object