Skip to content

Commit efd6045

Browse files
committed
fix test case because if a plan containes the exact same instance of Exchange multiple times then it makes no sense to replace one of the instances to a ReusedExchange
1 parent e6d4997 commit efd6045

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -470,7 +470,7 @@ class PlannerSuite extends SharedSparkSession {
470470
Inner,
471471
None,
472472
shuffle,
473-
shuffle)
473+
shuffle.copy())
474474

475475
val outputPlan = ReuseExchange(spark.sessionState.conf).apply(inputPlan)
476476
if (outputPlan.collect { case e: ReusedExchangeExec => true }.size != 1) {

0 commit comments

Comments
 (0)