xtern commented on code in PR #6356:
URL: https://github.com/apache/ignite-3/pull/6356#discussion_r2382665201
##########
modules/compatibility-tests/src/integrationTest/java/org/apache/ignite/internal/ItCompatibilityTest.java:
##########
@@ -49,29 +52,59 @@ protected void
configureInitParameters(InitParametersBuilder builder) {
@Override
protected void setupBaseVersion(Ignite baseIgnite) {
- sql(baseIgnite, "CREATE TABLE TEST(ID INT PRIMARY KEY, VAL VARCHAR)");
+ sql(baseIgnite, "CREATE TABLE TEST_ALL_TYPES("
+ + "ID INT PRIMARY KEY, "
+ + "VAL_INT INT, "
+ + "VAL_BIGINT BIGINT, "
+ + "VAL_FLOAT FLOAT, "
+ + "VAL_DOUBLE DOUBLE, "
+ + "VAL_DECIMAL DECIMAL(10,2), "
+ + "VAL_BOOL BOOLEAN, "
+ + "VAL_STR VARCHAR, "
+ + "VAL_DATE DATE, "
+ + "VAL_TIME TIME, "
+ + "VAL_TIMESTAMP TIMESTAMP)");
Review Comment:
what about "TIMESTAMP WITH LOCAL TIME ZONE" (aka ColumnType.TIMESTAMP)?
By the way, what is the plan if we add a new type in the new version?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
