From 07142cf6dc821d42d3357d0173ae2c6875c0f545 Mon Sep 17 00:00:00 2001
From: iamhumanbeing <iamhumanbeing@gmail.com>
Date: Sun, 27 Aug 2017 08:23:57 +0100
Subject: [PATCH] [SPARK-21843] testNameNote should be
"(minNumPostShufflePartitions: 5)"
Signed-off-by: iamhumanbeing <iamhumanbeinggmail.com>
## What changes were proposed in this pull request?
testNameNote = "(minNumPostShufflePartitions: 3) is not correct.
it should be "(minNumPostShufflePartitions: " + numPartitions + ")" in ExchangeCoordinatorSuite
## How was this patch tested?
unit tests
Please review http://spark.apache.org/contributing.html before opening a pull request.
Author: iamhumanbeing <iamhumanbeing@gmail.com>
Closes #19058 from iamhumanbeing/testnote.
---
.../apache/spark/sql/execution/ExchangeCoordinatorSuite.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/ExchangeCoordinatorSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/ExchangeCoordinatorSuite.scala
index 06bce9a240..ab4042a981 100644
--- a/sql/core/src/test/scala/org/apache/spark/sql/execution/ExchangeCoordinatorSuite.scala
+++ b/sql/core/src/test/scala/org/apache/spark/sql/execution/ExchangeCoordinatorSuite.scala
@@ -280,7 +280,7 @@ class ExchangeCoordinatorSuite extends SparkFunSuite with BeforeAndAfterAll {
Seq(Some(5), None).foreach { minNumPostShufflePartitions =>
val testNameNote = minNumPostShufflePartitions match {
- case Some(numPartitions) => "(minNumPostShufflePartitions: 3)"
+ case Some(numPartitions) => "(minNumPostShufflePartitions: " + numPartitions + ")"
case None => ""
}
--
GitLab