[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r813940508 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/QueryPlanDistinctKeys.scala ## @@ -0,0 +1,45 @@ +/* + * Licensed

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r812727258 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,118 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r812732998 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,118 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r812727258 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,118 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r812722470 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,118 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r812719768 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,118 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r740310943 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,100 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r740310943 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,100 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r740285441 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlanDistinctAttributes.scala ## @@ -0,0 +1,33 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r740285032 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,100 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r740284813 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,100 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r739663634 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,100 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r739663634 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/DistinctAttributesVisitor.scala ## @@ -0,0 +1,100 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r672388764 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveRedundantAggregatesInLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r672388764 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveRedundantAggregatesInLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r672388764 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveRedundantAggregatesInLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r672388764 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveRedundantAggregatesInLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r672016978 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveAggsThroughLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r672012328 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveAggsThroughLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r672012114 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveAggsThroughLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r672011852 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveAggsThroughLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r671763303 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveAggsThroughLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r671763262 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveAggsThroughLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r671762429 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveAggsThroughLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@ +/* + *

[GitHub] [spark] wangyum commented on a change in pull request #33404: [SPARK-36194][SQL] Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

wangyum commented on a change in pull request #33404: URL: https://github.com/apache/spark/pull/33404#discussion_r671761972 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RemoveAggsThroughLeftSemiAntiJoin.scala ## @@ -0,0 +1,40 @@ +/* + *