diff options
author | Yucai Yu <yucai.yu@intel.com> | 2016-03-15 21:44:58 -0700 |
---|---|---|
committer | Reynold Xin <rxin@databricks.com> | 2016-03-15 21:44:58 -0700 |
commit | 52b6a899be2711568d86ab99d1a2b4d1f9fef286 (patch) | |
tree | 02426b92b6aa2cd0e0bc2e09ea2e8b36de3a575b | |
parent | bbd887f53cc4fa03d97932e1b570bd7180783da5 (diff) | |
download | spark-52b6a899be2711568d86ab99d1a2b4d1f9fef286.tar.gz spark-52b6a899be2711568d86ab99d1a2b4d1f9fef286.tar.bz2 spark-52b6a899be2711568d86ab99d1a2b4d1f9fef286.zip |
[MINOR][TEST][SQL] Remove wrong "expected" parameter in checkNaNWithoutCodegen
## What changes were proposed in this pull request?
Remove the wrong "expected" parameter in MathFunctionsSuite.scala's checkNaNWithoutCodegen.
This function is to check NaN value, so the "expected" parameter is useless. The Callers do not pass "expected" value and the similar function like checkNaNWithGeneratedProjection and checkNaNWithOptimization do not use it also.
Author: Yucai Yu <yucai.yu@intel.com>
Closes #11718 from yucai/unused_expected.
-rw-r--r-- | sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala b/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala index fba5f53715..d6ac4040b7 100644 --- a/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala +++ b/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala @@ -122,7 +122,6 @@ class MathFunctionsSuite extends SparkFunSuite with ExpressionEvalHelper { private def checkNaNWithoutCodegen( expression: Expression, - expected: Any, inputRow: InternalRow = EmptyRow): Unit = { val actual = try evaluate(expression, inputRow) catch { case e: Exception => fail(s"Exception evaluating $expression", e) |