Repository: spark
Updated Branches:
  refs/heads/branch-2.2 144426cff -> f339e2fd7


[SPARK-22499][FOLLOWUP][SQL] Reduce input string expressions for Least and 
Greatest to reduce time in its test

## What changes were proposed in this pull request?

It's minor and trivial but looks 2000 input is good enough to reproduce and 
test in SPARK-22499.

## How was this patch tested?

Manually brought the change and tested.

Locally tested:

Before: 3m 21s 288ms
After: 1m 29s 134ms

Given the latest successful build took:

```
ArithmeticExpressionSuite:
- SPARK-22499: Least and greatest should not generate codes beyond 64KB (7 
minutes, 49 seconds)
```

I expect it's going to save 4ish mins.

Author: hyukjinkwon <gurwls...@apache.org>

Closes #21855 from HyukjinKwon/minor-fix-suite.

(cherry picked from commit 3d5c61e5fd24f07302e39b5d61294da79aa0c2f9)
Signed-off-by: hyukjinkwon <gurwls...@apache.org>


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

Branch: refs/heads/branch-2.2
Commit: f339e2fd7f9bfa1c7326749ad604e194ca0c300c
Parents: 144426c
Author: hyukjinkwon <gurwls...@apache.org>
Authored: Tue Jul 24 19:51:09 2018 +0800
Committer: hyukjinkwon <gurwls...@apache.org>
Committed: Tue Jul 24 19:51:53 2018 +0800

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


http://git-wip-us.apache.org/repos/asf/spark/blob/f339e2fd/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ArithmeticExpressionSuite.scala
----------------------------------------------------------------------
diff --git 
a/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ArithmeticExpressionSuite.scala
 
b/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ArithmeticExpressionSuite.scala
index ca66010..0676b94 100644
--- 
a/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ArithmeticExpressionSuite.scala
+++ 
b/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ArithmeticExpressionSuite.scala
@@ -329,7 +329,7 @@ class ArithmeticExpressionSuite extends SparkFunSuite with 
ExpressionEvalHelper
   }
 
   test("SPARK-22499: Least and greatest should not generate codes beyond 
64KB") {
-    val N = 3000
+    val N = 2000
     val strings = (1 to N).map(x => "s" * x)
     val inputsExpr = strings.map(Literal.create(_, StringType))
 


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

Reply via email to