fix: Fallback reverse function with array input

Author: cfmcgrady

spark/src/main/scala/org/apache/comet/serde/QueryPlanSerde.scala

@@ -915,6 +915,9 @@ object QueryPlanSerde extends Logging with CometExprShim {
       case l @ Length(child) if child.dataType == BinaryType =>
         withInfo(l, "Length on BinaryType is not supported")
         None
+      case r @ Reverse(child) if child.dataType.isInstanceOf[ArrayType] =>
+        withInfo(r, "Reverse on ArrayType is not supported")
+        None
       case expr =>
         QueryPlanSerde.exprSerdeMap.get(expr.getClass) match {
           case Some(handler) =>

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

@@ -708,4 +708,14 @@ class CometArrayExpressionSuite extends CometTestBase with AdaptiveSparkPlanHelp
       }
     }
   }
+
+  test("test reverse function") {
+    withTable("t1") {
+      sql(
+        "create table t1 using parquet as select sequence(id, id) as c1 from range(10)")
+      // FIXME: Change checkSparkAnswer to checkSparkAnswerAndOperator after resolving
+      //  https://github.com/apache/datafusion-comet/issues/2476
+      checkSparkAnswer("select reverse(c1) AS x FROM t1 ORDER BY c1")
+    }
+  }
 }