Skip to content

Commit 5d8af9c

Browse files
committed
[MINOR][VL] Re-enable stale ignored atan2 test in MathFunctionsValidateSuite
The atan2 test was marked ignore after a result mismatch (PR #1689, 2023) caused by Velox using std::atan2 directly. That was fixed upstream in oap-project/velox#263 which added sparksql::Atan2Function that mirrors Spark's Math.atan2(y + 0.0, x + 0.0) behaviour to normalise -0 inputs. The ignore was never removed after the fix landed. Also promotes MathFunctionsValidateSuite from abstract class to class (fixing the RAS-removal oversight from PR #11756, the same fix already applied to DateFunctionsValidateSuite) and disables ANSI mode for Spark 4, consistent with the approach in PR #12158 which adds further tests to this same suite.
1 parent 33be6fb commit 5d8af9c

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

backends-velox/src/test/scala/org/apache/gluten/functions/MathFunctionsValidateSuite.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ class MathFunctionsValidateSuiteAnsiOn extends FunctionsValidateSuite {
6666
}
6767
}
6868

69-
abstract class MathFunctionsValidateSuite extends FunctionsValidateSuite {
69+
class MathFunctionsValidateSuite extends FunctionsValidateSuite {
7070

7171
disableFallbackCheck
7272
import testImplicits._
@@ -96,7 +96,7 @@ abstract class MathFunctionsValidateSuite extends FunctionsValidateSuite {
9696
}
9797
}
9898

99-
ignore("atan2") {
99+
test("atan2") {
100100
runQueryAndCompare("SELECT atan2(double_field1, 0) from datatab limit 1") {
101101
checkGlutenPlan[ProjectExecTransformer]
102102
}

0 commit comments

Comments
 (0)