feat: Support Spark expression minutes_of_time

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

@@ -196,6 +196,7 @@ object QueryPlanSerde extends Logging with CometExprShim {
     classOf[LastDay] -> CometLastDay,
     classOf[Hour] -> CometHour,
     classOf[Minute] -> CometMinute,
+    classOf[MinutesOfTime] -> CometMinutesOfTime,
     classOf[Second] -> CometSecond,
     classOf[TruncDate] -> CometTruncDate,
     classOf[TruncTimestamp] -> CometTruncTimestamp,

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

@@ -21,7 +21,7 @@ package org.apache.comet.serde
 
 import java.util.Locale
 
-import org.apache.spark.sql.catalyst.expressions.{Attribute, DateAdd, DateDiff, DateFormatClass, DateSub, DayOfMonth, DayOfWeek, DayOfYear, GetDateField, Hour, LastDay, Literal, Minute, Month, Quarter, Second, TruncDate, TruncTimestamp, UnixDate, UnixTimestamp, WeekDay, WeekOfYear, Year}
+import org.apache.spark.sql.catalyst.expressions.{Attribute, DateAdd, DateDiff, DateFormatClass, DateSub, DayOfMonth, DayOfWeek, DayOfYear, GetDateField, Hour, LastDay, Literal, Minute, MinutesOfTime, Month, Quarter, Second, TruncDate, TruncTimestamp, UnixDate, UnixTimestamp, WeekDay, WeekOfYear, Year}
 import org.apache.spark.sql.types.{DateType, IntegerType, StringType, TimestampType}
 import org.apache.spark.unsafe.types.UTF8String
 
@@ -228,6 +228,32 @@ object CometMinute extends CometExpressionSerde[Minute] {
   }
 }
 
+object CometMinutesOfTime extends CometExpressionSerde[MinutesOfTime] {
+  override def convert(
+      expr: MinutesOfTime,
+      inputs: Seq[Attribute],
+      binding: Boolean): Option[ExprOuterClass.Expr] = {
+    val childExpr = exprToProtoInternal(expr.children.head, inputs, binding)
+
+    if (childExpr.isDefined) {
+      val builder = ExprOuterClass.Minute.newBuilder()
+      builder.setChild(childExpr.get)
+
+      // MinutesOfTime is a RuntimeReplaceable expression and doesn't have timeZoneId property.
+      builder.setTimezone("UTC")
+
+      Some(
+        ExprOuterClass.Expr
+          .newBuilder()
+          .setMinute(builder)
+          .build())
+    } else {
+      withInfo(expr, expr.children.head)
+      None
+    }
+  }
+}
+
 object CometSecond extends CometExpressionSerde[Second] {
   override def convert(
       expr: Second,

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

@@ -618,6 +618,21 @@ class CometExpressionSuite extends CometTestBase with AdaptiveSparkPlanHelper {
     }
   }
 
+  test("MinutesOfTime expression support") {
+    Seq(true, false).foreach { dictionaryEnabled =>
+      withTempDir { dir =>
+        val path = new Path(dir.toURI.toString, "part-r-0.parquet")
+        makeRawTimeParquetFile(path, dictionaryEnabled = dictionaryEnabled, 10000)
+        readParquetFile(path.toString) { df =>
+          // Test that MinutesOfTime (via minute() function) works correctly
+          val query = df.select(expr("minute(_1)"))
+
+          checkSparkAnswerAndOperator(query)
+        }
+      }
+    }
+  }
+
   test("hour on int96 timestamp column") {
     import testImplicits._