Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19873#discussion_r154855592
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala ---
@@ -241,7 +241,7 @@ class PlannerSuite extends SharedSQLContext {
test("collapse adjacent repartitions") {
val doubleRepartitioned =
testData.repartition(10).repartition(20).coalesce(5)
def countRepartitions(plan: LogicalPlan): Int = plan.collect { case r:
Repartition => r }.length
- assert(countRepartitions(doubleRepartitioned.queryExecution.logical)
=== 3)
+ assert(countRepartitions(doubleRepartitioned.queryExecution.analyzed)
=== 3)
--- End diff --
is it a necessary change?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
