spark git commit: [SPARK-23928][TESTS][FOLLOWUP] Set seed to avoid flakiness

[gu...@apache.org]

Repository: spark
Updated Branches:
  refs/heads/master b4bf8be54 -> 6a143e3eb


[SPARK-23928][TESTS][FOLLOWUP] Set seed to avoid flakiness

## What changes were proposed in this pull request?

The tests for shuffle can be flaky (eg. https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94355/testReport/). This happens because we have not set the seed for `Random`.

## How was this patch tested?

running 10000 times the UT (validated that with a different seed eg. 12345 the test fails).

Closes #22023 from mgaido91/SPARK-23928_followup.

Authored-by: Marco Gaido <ma...@gmail.com>
Signed-off-by: hyukjinkwon <gu...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/6a143e3e
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/6a143e3e
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/6a143e3e

Branch: refs/heads/master
Commit: 6a143e3ebf4bcac9464561e67174a7c610d9be1f
Parents: b4bf8be
Author: Marco Gaido <ma...@gmail.com>
Authored: Tue Aug 7 22:23:59 2018 +0800
Committer: hyukjinkwon <gu...@apache.org>
Committed: Tue Aug 7 22:23:59 2018 +0800

----------------------------------------------------------------------
 .../sql/catalyst/expressions/CollectionExpressionsSuite.scala      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/6a143e3e/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
----------------------------------------------------------------------
diff --git a/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala b/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
index c6b3f95..40487b3 100644
--- a/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
+++ b/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
@@ -1489,7 +1489,7 @@ class CollectionExpressionsSuite extends SparkFunSuite with ExpressionEvalHelper
     checkEvaluation(Shuffle(as7, Some(0)), null)
     checkEvaluation(Shuffle(aa, Some(0)), Seq(Seq("e"), Seq("a", "b"), Seq("c", "d")))
 
-    val r = new Random()
+    val r = new Random(1234)
     val seed1 = Some(r.nextLong())
     assert(evaluateWithoutCodegen(Shuffle(ai0, seed1)) ===
       evaluateWithoutCodegen(Shuffle(ai0, seed1)))


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org