[GitHub] [spark] tanelk commented on a change in pull request #32904: [SPARK-33122][FOLLOW-UP][SQL] Distinct aggs are not duplicate sensitive

2021-06-15 Thread GitBox
tanelk commented on a change in pull request #32904: URL: https://github.com/apache/spark/pull/32904#discussion_r651689815 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveRedundantAggregates.scala ## @@ -67,4 +67,8 @@ object RemoveRedund

[GitHub] [spark] tanelk commented on a change in pull request #32904: [SPARK-33122][FOLLOW-UP][SQL] Distinct aggs are not duplicate sensitive

2021-06-15 Thread GitBox
tanelk commented on a change in pull request #32904: URL: https://github.com/apache/spark/pull/32904#discussion_r651693288 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveRedundantAggregates.scala ## @@ -67,4 +67,8 @@ object RemoveRedund

[GitHub] [spark] tanelk commented on a change in pull request #32904: [SPARK-33122][FOLLOW-UP][SQL] Distinct aggs are not duplicate sensitive

2021-06-15 Thread GitBox
tanelk commented on a change in pull request #32904: URL: https://github.com/apache/spark/pull/32904#discussion_r651693288 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveRedundantAggregates.scala ## @@ -67,4 +67,8 @@ object RemoveRedund

[GitHub] [spark] tanelk commented on a change in pull request #32904: [SPARK-33122][FOLLOW-UP][SQL] Distinct aggs are not duplicate sensitive

2021-06-15 Thread GitBox
tanelk commented on a change in pull request #32904: URL: https://github.com/apache/spark/pull/32904#discussion_r651701796 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveRedundantAggregates.scala ## @@ -67,4 +67,8 @@ object RemoveRedund

[GitHub] [spark] tanelk commented on a change in pull request #32904: [SPARK-33122][FOLLOW-UP][SQL] Distinct aggs are not duplicate sensitive

2021-06-15 Thread GitBox
tanelk commented on a change in pull request #32904: URL: https://github.com/apache/spark/pull/32904#discussion_r651701796 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveRedundantAggregates.scala ## @@ -67,4 +67,8 @@ object RemoveRedund