fix: update

Signed-off-by: zacsun <[email protected]>

Author: predator4ann

api/src/main/java/org/apache/gravitino/rel/types/Types.java

@@ -364,7 +364,7 @@ public Name name() {
 
     @Override
     public String simpleString() {
-      return precision.map(integer -> String.format("time(%d)", integer)).orElse("time");
+      return precision.filter(p -> p > 0).map(p -> String.format("time(%d)", p)).orElse("time");
     }
 
     @Override
@@ -441,11 +441,12 @@ public Name name() {
     @Override
     public String simpleString() {
       return precision
+          .filter(p -> p > 0)
           .map(
-              integer ->
+              p ->
                   withTimeZone
-                      ? String.format("timestamp_tz(%d)", integer)
-                      : String.format("timestamp(%d)", integer))
+                      ? String.format("timestamp_tz(%d)", p)
+                      : String.format("timestamp(%d)", p))
           .orElse(withTimeZone ? "timestamp_tz" : "timestamp");
     }
 

catalogs/catalog-jdbc-mysql/src/main/java/org/apache/gravitino/catalog/mysql/converter/MysqlTypeConverter.java

@@ -138,9 +138,11 @@ public String fromGravitino(Type type) {
       // such as DATETIME.) see more details: https://dev.mysql.com/doc/refman/8.0/en/datetime.html
       Types.TimestampType timestampType = (Types.TimestampType) type;
       String baseType = timestampType.hasTimeZone() ? TIMESTAMP : DATETIME;
-      return timestampType.precision().isPresent()
-          ? String.format("%s(%d)", baseType, timestampType.precision().get())
-          : baseType;
+      return timestampType
+          .precision()
+          .filter(p -> p > 0)
+          .map(p -> String.format("%s(%d)", baseType, p))
+          .orElse(baseType);
     } else if (type instanceof Types.DecimalType) {
       return type.simpleString();
     } else if (type instanceof Types.VarCharType) {

catalogs/catalog-jdbc-mysql/src/main/java/org/apache/gravitino/catalog/mysql/operation/MysqlTableOperations.java

@@ -679,9 +679,9 @@ private static void validateIndexes(Index[] indexes, JdbcColumn[] columns) {
    * TIME: Format is 'HH:MM:SS' (8 chars) + precision - TIMESTAMP/DATETIME: Format is 'YYYY-MM-DD
    * HH:MM:SS' (19 chars) + precision
    *
-   * <p>The precision is calculated by subtracting the base format length from the total column
-   * size. For example: - TIME(6) has columnSize = 14 (8 + 6) - TIMESTAMP(6) has columnSize = 25 (19
-   * + 6)
+   * <p>The precision is calculated by subtracting the base format length plus one (for decimal
+   * point) from the total column size. For example: - TIME(6) has columnSize = 15 (8 + 1 + 6) -
+   * TIMESTAMP(6) has columnSize = 26 (19 + 1 + 6)
    *
    * @param typeName The data type name (TIME, TIMESTAMP, or DATETIME)
    * @param columnSize The total column size from MySQL JDBC driver
@@ -692,12 +692,12 @@ public int calculateDatetimePrecision(String typeName, int columnSize) {
     String upperTypeName = typeName.toUpperCase();
     switch (upperTypeName) {
       case "TIME":
-        // TIME format: 'HH:MM:SS' (8 chars) + precision
-        return columnSize > 8 ? columnSize - 9 : 0;
+        // TIME format: 'HH:MM:SS' (8 chars) + decimal point + precision
+        return columnSize >= 9 ? columnSize - 9 : 0;
       case "TIMESTAMP":
       case "DATETIME":
-        // TIMESTAMP/DATETIME format: 'YYYY-MM-DD HH:MM:SS' (19 chars) + precision
-        return columnSize > 19 ? columnSize - 20 : 0;
+        // TIMESTAMP/DATETIME format: 'YYYY-MM-DD HH:MM:SS' (19 chars) + decimal point + precision
+        return columnSize >= 20 ? columnSize - 20 : 0;
     }
     return 0;
   }