fix build break

Author: davidm-db

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystTypeConverters.scala

@@ -24,12 +24,14 @@ import java.sql.{Date, Timestamp}
 import java.time.{Duration, Instant, LocalDate, LocalDateTime, LocalTime, Period}
 import java.util.{Map => JavaMap}
 import javax.annotation.Nullable
+
 import scala.language.existentials
+
 import org.apache.spark.SparkIllegalArgumentException
 import org.apache.spark.sql.Row
 import org.apache.spark.sql.catalyst.expressions._
 import org.apache.spark.sql.catalyst.util._
-import org.apache.spark.sql.errors.{QueryCompilationErrors, QueryExecutionErrors}
+import org.apache.spark.sql.errors.QueryCompilationErrors
 import org.apache.spark.sql.internal.SQLConf
 import org.apache.spark.sql.types._
 import org.apache.spark.sql.types.DayTimeIntervalType._

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Cast.scala

@@ -611,12 +611,14 @@ case class Cast(
         messageParameters = Map("other" -> other.toString))
     }
     if (canCast) {
-      // If the cast is to a time type, we first need to check if the time type is enabled.
-      case _: TimeType if !SQLConf.get.isTimeTypeEnabled =>
-        throw QueryCompilationErrors.unsupportedTimeTypeError()
-      // If the cast is not to a time type, we can skip the time type check and continue.
-      case _ =>
-        TypeCheckResult.TypeCheckSuccess
+      dataType match {
+        // If the cast is to a time type, we first need to check if the time type is enabled.
+        case _: TimeType if !SQLConf.get.isTimeTypeEnabled =>
+          throw QueryCompilationErrors.unsupportedTimeTypeError()
+        // If the cast is not to a time type, we can skip the time type check and continue.
+        case _ =>
+          TypeCheckResult.TypeCheckSuccess
+      }
     } else {
       typeCheckFailureInCast
     }