zml1206 commented on code in PR #9848:
URL: https://github.com/apache/incubator-gluten/pull/9848#discussion_r2123111837
##########
backends-velox/src/test/scala/org/apache/gluten/functions/MathFunctionsValidateSuite.scala:
##########
@@ -217,47 +217,47 @@ abstract class MathFunctionsValidateSuite extends
FunctionsValidateSuite {
}
}
- test("Test log function") {
+ test("log function") {
runQueryAndCompare("SELECT log(10, l_orderkey) from lineitem limit 1") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
- test("Test log10 function") {
+ test("log10 function") {
runQueryAndCompare("SELECT log10(l_orderkey) from lineitem limit 1") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
- test("Test negative function") {
+ test("negative function") {
runQueryAndCompare("select negative(l_orderkey) from lineitem") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
- test("Test pmod function") {
+ test("pmod") {
val df = runQueryAndCompare("SELECT pmod(cast(l_orderkey as int), 3) from
lineitem limit 1") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
checkLengthAndPlan(df, 1)
}
- test("Test power function") {
+ test("power function") {
Review Comment:
ditto
##########
backends-velox/src/test/scala/org/apache/gluten/functions/MathFunctionsValidateSuite.scala:
##########
@@ -217,47 +217,47 @@ abstract class MathFunctionsValidateSuite extends
FunctionsValidateSuite {
}
}
- test("Test log function") {
+ test("log function") {
runQueryAndCompare("SELECT log(10, l_orderkey) from lineitem limit 1") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
- test("Test log10 function") {
+ test("log10 function") {
runQueryAndCompare("SELECT log10(l_orderkey) from lineitem limit 1") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
- test("Test negative function") {
+ test("negative function") {
runQueryAndCompare("select negative(l_orderkey) from lineitem") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
- test("Test pmod function") {
+ test("pmod") {
val df = runQueryAndCompare("SELECT pmod(cast(l_orderkey as int), 3) from
lineitem limit 1") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
checkLengthAndPlan(df, 1)
}
- test("Test power function") {
+ test("power function") {
val df = runQueryAndCompare("SELECT power(l_orderkey, 2) from lineitem
limit 1") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
checkLengthAndPlan(df, 1)
}
- test("Test rand function") {
+ test("rand function") {
Review Comment:
ditto
##########
backends-velox/src/test/scala/org/apache/gluten/functions/MathFunctionsValidateSuite.scala:
##########
@@ -288,51 +288,51 @@ abstract class MathFunctionsValidateSuite extends
FunctionsValidateSuite {
compareResultsAgainstVanillaSpark("select round(44, -1)", true, { _ => })
}
- test("Test shiftleft function") {
+ test("shiftleft function") {
runQueryAndCompare("SELECT shiftleft(int_field1, 1) from datatab limit 1")
{
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
- test("Test try_add function") {
+ test("try_add function") {
runQueryAndCompare(
"select try_add(cast(l_orderkey as int), 1), try_add(cast(l_orderkey as
int), 2147483647)" +
" from lineitem") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
- test("Test try_divide function") {
+ test("try_divide function") {
runQueryAndCompare(
"select try_divide(cast(l_orderkey as int), 0) from lineitem",
noFallBack = false) {
_ => // Spark would always cast inputs to double for this function.
}
}
- testWithMinSparkVersion("Test try_multiply function", "3.3") {
+ testWithMinSparkVersion("try_multiply function", "3.3") {
runQueryAndCompare(
"select try_multiply(2147483647, cast(l_orderkey as int)), " +
"try_multiply(-2147483648, cast(l_orderkey as int)) from lineitem") {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
- testWithMinSparkVersion("Test try_subtract function", "3.3") {
+ testWithMinSparkVersion("try_subtract function", "3.3") {
Review Comment:
ditto
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
