Enable ABS tests in legacy/ANSI mode

Author: hsiang-c

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

@@ -1385,8 +1385,7 @@ class CometExpressionSuite extends CometTestBase with AdaptiveSparkPlanHelper {
     testDoubleScalarExpr("expm1")
   }
 
-  // https://github.com/apache/datafusion-comet/issues/666
-  ignore("abs") {
+  test("abs") {
     Seq(true, false).foreach { dictionaryEnabled =>
       withTempDir { dir =>
         val path = new Path(dir.toURI.toString, "test.parquet")
@@ -1400,8 +1399,7 @@ class CometExpressionSuite extends CometTestBase with AdaptiveSparkPlanHelper {
     }
   }
 
-  // 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 +1432,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));