Improve handling of negative GRACE PERIOD

Previously, providing a negative grace period resulted in a low-level
internal exception:

Caused by: java.lang.IllegalArgumentException: gracePeriod cannot be negative: Optional[PT-1H]
	at com.google.common.base.Preconditions.checkArgument(Preconditions.java:217)
	at io.trino.metadata.MaterializedViewDefinition.<init>(MaterializedViewDefinition.java:49)

Author: piotrrzysko

core/trino-parser/src/main/java/io/trino/sql/parser/AstBuilder.java

@@ -610,7 +610,11 @@ public Node visitCreateMaterializedView(SqlBaseParser.CreateMaterializedViewCont
     {
         Optional<IntervalLiteral> gracePeriod = Optional.empty();
         if (context.GRACE() != null) {
-            gracePeriod = Optional.of((IntervalLiteral) visit(context.interval()));
+            IntervalLiteral interval = (IntervalLiteral) visit(context.interval());
+            if (interval.getSign() == IntervalLiteral.Sign.NEGATIVE) {
+                throw parseError("GRACE PERIOD cannot be negative", context.interval());
+            }
+            gracePeriod = Optional.of(interval);
         }
 
         Optional<String> comment = Optional.empty();

core/trino-parser/src/test/java/io/trino/sql/parser/TestSqlParser.java

@@ -5973,6 +5973,10 @@ public void testCreateMaterializedView()
                         ImmutableList.of(),
                         Optional.empty()));
 
+        assertThatThrownBy(() -> SQL_PARSER.createStatement("CREATE MATERIALIZED VIEW a GRACE PERIOD INTERVAL -'2' DAY AS SELECT * FROM t"))
+                .isInstanceOf(ParsingException.class)
+                .hasMessageMatching("line 1:41: GRACE PERIOD cannot be negative");
+
         // OR REPLACE, COMMENT, WITH properties
         assertThat(statement(
                 """

testing/trino-testing/src/main/java/io/trino/testing/BaseConnectorTest.java

@@ -1511,6 +1511,10 @@ public void testMaterializedViewGracePeriod()
             return;
         }
 
+        assertQueryFails(
+                "CREATE MATERIALIZED VIEW " + viewName + " GRACE PERIOD INTERVAL -'1' HOUR AS SELECT * FROM nation",
+                "line 1:71: GRACE PERIOD cannot be negative");
+
         try (TestTable table = newTrinoTable(
                 "test_base_table",
                 "AS TABLE region")) {