From 8dfd46d42b16799ca8793a9862ef55d1ff9913d3 Mon Sep 17 00:00:00 2001 From: Rui Mo Date: Tue, 23 Sep 2025 17:19:50 +0100 Subject: [PATCH] Enable datetime-legacy.sql --- .../sql-tests/results/datetime-legacy.sql.out | 8 ++++---- .../org/apache/spark/sql/GlutenSQLQueryTestSuite.scala | 3 +-- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/gluten-ut/spark35/src/test/resources/backends-velox/sql-tests/results/datetime-legacy.sql.out b/gluten-ut/spark35/src/test/resources/backends-velox/sql-tests/results/datetime-legacy.sql.out index 5e2a4d61e118..678ec166605a 100644 --- a/gluten-ut/spark35/src/test/resources/backends-velox/sql-tests/results/datetime-legacy.sql.out +++ b/gluten-ut/spark35/src/test/resources/backends-velox/sql-tests/results/datetime-legacy.sql.out @@ -1398,8 +1398,8 @@ select TIMESTAMP_SECONDS(1230219000123123) -- !query schema struct<> -- !query output -java.lang.ArithmeticException -long overflow +org.apache.gluten.exception.GlutenException +Could not convert Timestamp(1230219000123123, 0) to microseconds -- !query @@ -1407,8 +1407,8 @@ select TIMESTAMP_SECONDS(-1230219000123123) -- !query schema struct<> -- !query output -java.lang.ArithmeticException -long overflow +org.apache.gluten.exception.GlutenException +Could not convert Timestamp(-1230219000123123, 0) to microseconds -- !query diff --git a/gluten-ut/spark35/src/test/scala/org/apache/spark/sql/GlutenSQLQueryTestSuite.scala b/gluten-ut/spark35/src/test/scala/org/apache/spark/sql/GlutenSQLQueryTestSuite.scala index 2d721e394db1..6056ea2ac9f5 100644 --- a/gluten-ut/spark35/src/test/scala/org/apache/spark/sql/GlutenSQLQueryTestSuite.scala +++ b/gluten-ut/spark35/src/test/scala/org/apache/spark/sql/GlutenSQLQueryTestSuite.scala @@ -141,8 +141,7 @@ class GlutenSQLQueryTestSuite "ignored.sql", // Do NOT remove this one. It is here to test the ignore functionality. "explain-aqe.sql", // Explain is different in Gluten. "explain-cbo.sql", // Explain is different in Gluten. - "explain.sql", // Explain is different in Gluten. - "datetime-legacy.sql" // https://github.com/apache/incubator-gluten/issues/10671 + "explain.sql" // Explain is different in Gluten. ) ++ otherIgnoreList ++ BackendTestSettings.instance.getSQLQueryTestSettings.getIgnoredSQLQueryTests