This is an automated email from the ASF dual-hosted git repository. changchen pushed a commit to branch feature/ansi-ci-baseline in repository https://gitbox.apache.org/repos/asf/gluten.git
commit d68972959b6a6ba185fbaea4dca0c1e5a336c480 Author: Chang chen <[email protected]> AuthorDate: Sat Apr 11 03:08:16 2026 +0000 [GLUTEN-10134][VL] Fix CI Run 10: filter Coyhaique timezone in spark40 TryCastSuite and skip corr() ANSI test - Add America/Coyhaique timezone filter to spark40 GlutenTryCastSuite (sync with spark41) - Skip "Support multi-children count with row construct" under ANSI mode: Velox corr() returns NaN on DIVIDE_BY_ZERO instead of throwing Co-Authored-By: Claude Opus 4 <[email protected]> --- .../test/scala/org/apache/gluten/execution/MiscOperatorSuite.scala | 4 +++- .../apache/spark/sql/catalyst/expressions/GlutenTryCastSuite.scala | 7 ++++++- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/backends-velox/src/test/scala/org/apache/gluten/execution/MiscOperatorSuite.scala b/backends-velox/src/test/scala/org/apache/gluten/execution/MiscOperatorSuite.scala index 9fbd99752e..22477ef271 100644 --- a/backends-velox/src/test/scala/org/apache/gluten/execution/MiscOperatorSuite.scala +++ b/backends-velox/src/test/scala/org/apache/gluten/execution/MiscOperatorSuite.scala @@ -1615,7 +1615,9 @@ class MiscOperatorSuite extends VeloxWholeStageTransformerSuite with AdaptiveSpa )(df => checkFallbackOperators(df, 0)) } - test("Support multi-children count with row construct") { + // corr() triggers DIVIDE_BY_ZERO in ANSI mode when group has single row (n=1, stddev=0). + // Velox's corr() returns NaN instead of throwing. Skip under ANSI mode. + testWithAnsiMode("Support multi-children count with row construct", ansiEnabled = false) { runQueryAndCompare( """ |select l_orderkey, count(distinct l_partkey, l_comment), corr(l_partkey, l_partkey+1) diff --git a/gluten-ut/spark40/src/test/scala/org/apache/spark/sql/catalyst/expressions/GlutenTryCastSuite.scala b/gluten-ut/spark40/src/test/scala/org/apache/spark/sql/catalyst/expressions/GlutenTryCastSuite.scala index fc15ebfeef..c51980ecaa 100644 --- a/gluten-ut/spark40/src/test/scala/org/apache/spark/sql/catalyst/expressions/GlutenTryCastSuite.scala +++ b/gluten-ut/spark40/src/test/scala/org/apache/spark/sql/catalyst/expressions/GlutenTryCastSuite.scala @@ -46,7 +46,11 @@ class GlutenTryCastSuite extends TryCastSuite with GlutenTestsTrait { // SystemV timezones are a legacy way of specifying timezones in Unix-like OS. // It is not supported by Velox. - for (tz <- ALL_TIMEZONES.filterNot(_.getId.contains("SystemV"))) { + for ( + tz <- ALL_TIMEZONES + .filterNot(_.getId.contains("SystemV")) + .filterNot(_.getId.contains("America/Coyhaique")) + ) { withSQLConf( SQLConf.SESSION_LOCAL_TIMEZONE.key -> tz.getId ) { @@ -118,6 +122,7 @@ class GlutenTryCastSuite extends TryCastSuite with GlutenTestsTrait { .filterNot(_.getId.contains("SystemV")) .filterNot(_.getId.contains("Europe/Kyiv")) .filterNot(_.getId.contains("America/Ciudad_Juarez")) + .filterNot(_.getId.contains("America/Coyhaique")) .filterNot(_.getId.contains("Antarctica/Vostok")) .filterNot(_.getId.contains("Pacific/Kanton")) .filterNot(_.getId.contains("Asia/Tehran")) --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
