[GitHub] [spark] squito commented on a change in pull request #28885: [SPARK-29375][SPARK-28940][SPARK-32041][SQL] Whole plan exchange and subquery reuse

squito commented on a change in pull request #28885: URL: https://github.com/apache/spark/pull/28885#discussion_r465997277 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/ExchangeSuite.scala ## @@ -156,4 +158,46 @@ class ExchangeSuite extends

[GitHub] [spark] squito commented on a change in pull request #28885: [SPARK-29375][SPARK-28940][SPARK-32041][SQL] Whole plan exchange and subquery reuse

squito commented on a change in pull request #28885: URL: https://github.com/apache/spark/pull/28885#discussion_r460099447 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/ExchangeSuite.scala ## @@ -156,4 +158,46 @@ class ExchangeSuite extends

[GitHub] [spark] squito commented on a change in pull request #28885: [SPARK-29375][SPARK-28940][SPARK-32041][SQL] Whole plan exchange and subquery reuse

squito commented on a change in pull request #28885: URL: https://github.com/apache/spark/pull/28885#discussion_r459091567 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/Exchange.scala ## @@ -95,46 +89,3 @@ case class ReusedExchangeExec(override

[GitHub] [spark] squito commented on a change in pull request #28885: [SPARK-29375][SPARK-28940][SPARK-32041][SQL] Whole plan exchange and subquery reuse

squito commented on a change in pull request #28885: URL: https://github.com/apache/spark/pull/28885#discussion_r459072387 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/ReuseExchangeAndSubquerySuite.scala ## @@ -0,0 +1,110 @@ +/* + * Licensed to the