fix(trino): update timezone grammar to avoid ambiguity (#394)

* fix(trino): update timezone grammar to avoid ambiguity

* test(trino): add semicolon for test sqls

Author: JackWang032

src/grammar/trino/TrinoSql.g4

@@ -737,17 +737,15 @@ normalForm
     ;
 
 type
-    : KW_ROW '(' rowField (',' rowField)* ')'                                             # rowType
-    | KW_INTERVAL from=intervalField (KW_TO to=intervalField)?                            # intervalType
-    | base=KW_TIMESTAMP ('(' precision= typeParameter ')')? (KW_WITHOUT KW_TIME KW_ZONE)? # dateTimeType
-    | base=KW_TIMESTAMP ('(' precision= typeParameter ')')? KW_WITH KW_TIME KW_ZONE       # dateTimeType
-    | base=KW_TIME ('(' precision= typeParameter ')')? (KW_WITHOUT KW_TIME KW_ZONE)?      # dateTimeType
-    | base=KW_TIME ('(' precision= typeParameter ')')? KW_WITH KW_TIME KW_ZONE            # dateTimeType
-    | KW_DOUBLE KW_PRECISION                                                              # doublePrecisionType
-    | KW_ARRAY '<' type '>'                                                               # legacyArrayType
-    | KW_MAP '<' keyType=type ',' valueType=type '>'                                      # legacyMapType
-    | type KW_ARRAY ('[' INTEGER_VALUE ']')?                                              # arrayType
-    | identifier ('(' typeParameter (',' typeParameter)* ')')?                            # genericType
+    : KW_ROW '(' rowField (',' rowField)* ')'                                                         # rowType
+    | KW_INTERVAL from=intervalField (KW_TO to=intervalField)?                                        # intervalType
+    | base=KW_TIMESTAMP ('(' precision= typeParameter ')')? ((KW_WITH | KW_WITHOUT) KW_TIME KW_ZONE)? # dateTimeType
+    | base=KW_TIME ('(' precision= typeParameter ')')? ((KW_WITH | KW_WITHOUT) KW_TIME KW_ZONE)?      # timeType
+    | KW_DOUBLE KW_PRECISION                                                                          # doublePrecisionType
+    | KW_ARRAY '<' type '>'                                                                           # legacyArrayType
+    | KW_MAP '<' keyType=type ',' valueType=type '>'                                                  # legacyMapType
+    | type KW_ARRAY ('[' INTEGER_VALUE ']')?                                                          # arrayType
+    | identifier ('(' typeParameter (',' typeParameter)* ')')?                                        # genericType
     ;
 
 rowField

src/lib/trino/TrinoSql.interp

@@ -268,6 +268,7 @@ import { RowTypeContext } from "./TrinoSqlParser.js";
 import { IntervalTypeContext } from "./TrinoSqlParser.js";
 import { ArrayTypeContext } from "./TrinoSqlParser.js";
 import { DoublePrecisionTypeContext } from "./TrinoSqlParser.js";
+import { TimeTypeContext } from "./TrinoSqlParser.js";
 import { LegacyArrayTypeContext } from "./TrinoSqlParser.js";
 import { GenericTypeContext } from "./TrinoSqlParser.js";
 import { DateTimeTypeContext } from "./TrinoSqlParser.js";
@@ -3356,6 +3357,18 @@ export class TrinoSqlListener implements ParseTreeListener {
      * @param ctx the parse tree
      */
     exitDoublePrecisionType?: (ctx: DoublePrecisionTypeContext) => void;
+    /**
+     * Enter a parse tree produced by the `timeType`
+     * labeled alternative in `TrinoSqlParser.type`.
+     * @param ctx the parse tree
+     */
+    enterTimeType?: (ctx: TimeTypeContext) => void;
+    /**
+     * Exit a parse tree produced by the `timeType`
+     * labeled alternative in `TrinoSqlParser.type`.
+     * @param ctx the parse tree
+     */
+    exitTimeType?: (ctx: TimeTypeContext) => void;
     /**
      * Enter a parse tree produced by the `legacyArrayType`
      * labeled alternative in `TrinoSqlParser.type`.