xtern commented on code in PR #5490:
URL: https://github.com/apache/ignite-3/pull/5490#discussion_r2013590054
##########
modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/sql/SqlDdlParserTest.java:
##########
@@ -767,15 +770,36 @@ public void dropIndexIfExists() {
}
/**
- * Ensures that the user cannot use the TIME_WITH_LOCAL_TIME_ZONE and
TIMESTAMP_WITH_LOCAL_TIME_ZONE types for table columns.
+ * Ensures that the parser throws the expected exception when attempting
to use the following unsupported types for table columns.
+ *
+ * <ol>
+ * <li>{@link SqlTypeName#TIME_TZ TIME WITH TIME ZONE}</li>
+ * <li>{@link SqlTypeName#TIME_WITH_LOCAL_TIME_ZONE TIME WITH LOCAL
TIME ZONE}</li>
+ * <li>{@link SqlTypeName#TIMESTAMP_TZ TIMESTAMP WITH TIME ZONE}</li>
+ * </ol>
*/
// TODO: Remove after https://issues.apache.org/jira/browse/IGNITE-21555
is implemented.
- @Test
- public void timeWithLocalTimeZoneIsNotSupported() {
+ @ParameterizedTest(name = "ензу={0}")
+ @CsvSource({
+ "TIME WITH TIME ZONE, Encountered \"WITH\"",
+ "TIME WITH LOCAL TIME ZONE, Encountered \"WITH\"",
+ "TIME WITHOUT TIME ZONE, Encountered \"WITHOUT\"",
+ "TIME WITHOUT LOCAL TIME ZONE, Encountered \"WITHOUT\"",
+ "TIMESTAMP WITH TIME ZONE, Encountered \"TIME\"",
+ "TIMESTAMP WITHOUT TIME ZONE, Encountered \"WITHOUT\"",
Review Comment:
I thought that since we don't support "WITH TIME ZONE" we should not support
"WITHOUT TIME ZONE"
```
Without Feature F411, “Time zone specification”, conforming SQL language
shall not contain <with or
without time zone>.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
