Enable ABS tests in legacy/ANSI mode

Author: hsiang-c

spark/src/test/scala/org/apache/comet/CometExpressionSuite.scala

@@ -1385,23 +1385,25 @@ class CometExpressionSuite extends CometTestBase with AdaptiveSparkPlanHelper {
     testDoubleScalarExpr("expm1")
   }
 
-  // https://github.com/apache/datafusion-comet/issues/666
-  ignore("abs") {
-    Seq(true, false).foreach { dictionaryEnabled =>
-      withTempDir { dir =>
-        val path = new Path(dir.toURI.toString, "test.parquet")
-        makeParquetFileAllPrimitiveTypes(path, dictionaryEnabled = dictionaryEnabled, 100)
-        withParquetTable(path.toString, "tbl") {
-          Seq(2, 3, 4, 5, 6, 7, 9, 10, 11, 12, 15, 16, 17).foreach { col =>
-            checkSparkAnswerAndOperator(s"SELECT abs(_${col}) FROM tbl")
+  test("abs") {
+    Seq(true, false).foreach { ansi_enabled =>
+      Seq(true, false).foreach { dictionaryEnabled =>
+        withSQLConf(SQLConf.ANSI_ENABLED.key -> ansi_enabled.toString) {
+          withTempDir { dir =>
+            val path = new Path(dir.toURI.toString, "test.parquet")
+            makeParquetFileAllPrimitiveTypes(path, dictionaryEnabled = dictionaryEnabled, 100)
+            withParquetTable(path.toString, "tbl") {
+              Seq(2, 3, 4, 5, 6, 7, 9, 10, 11, 12, 15, 16, 17).foreach { col =>
+                checkSparkAnswerAndOperator(s"SELECT abs(_${col}) FROM tbl")
+              }
+            }
           }
         }
       }
     }
   }
 
-  // https://github.com/apache/datafusion-comet/issues/666
-  ignore("abs Overflow ansi mode") {
+  test("abs Overflow ANSI mode") {
 
     def testAbsAnsiOverflow[T <: Product: ClassTag: TypeTag](data: Seq[T]): Unit = {
       withParquetTable(data, "tbl") {
@@ -1434,8 +1436,7 @@ class CometExpressionSuite extends CometTestBase with AdaptiveSparkPlanHelper {
     }
   }
 
-  // https://github.com/apache/datafusion-comet/issues/666
-  ignore("abs Overflow legacy mode") {
+  test("abs Overflow legacy mode") {
 
     def testAbsLegacyOverflow[T <: Product: ClassTag: TypeTag](data: Seq[T]): Unit = {
       withSQLConf(SQLConf.ANSI_ENABLED.key -> "false") {

spark/src/test/scala/org/apache/spark/sql/CometTestBase.scala

@@ -504,10 +504,10 @@ abstract class CometTestBase
        |  optional float                    _6;
        |  optional double                   _7;
        |  optional binary                   _8(UTF8);
-       |  optional int32                    _9(UINT_8);
-       |  optional int32                    _10(UINT_16);
-       |  optional int32                    _11(UINT_32);
-       |  optional int64                    _12(UINT_64);
+       |  optional int32                    _9(INT_8);
+       |  optional int32                    _10(INT_16);
+       |  optional int32                    _11(INT_32);
+       |  optional int64                    _12(INT_64);
        |  optional binary                   _13(ENUM);
        |  optional FIXED_LEN_BYTE_ARRAY(3)  _14;
        |  optional int32                    _15(DECIMAL(5, 2));