Skip to content

Commit 50a87c7

Browse files
committed
Add temporal data tests with INSERT in Presto
1 parent 65880d9 commit 50a87c7

File tree

3 files changed

+25
-0
lines changed

3 files changed

+25
-0
lines changed

presto-mysql/src/test/java/io/prestosql/plugin/mysql/TestMySqlTypeMapping.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -429,6 +429,8 @@ public void testDate()
429429
.build();
430430
testCases.execute(getQueryRunner(), session, mysqlCreateAndInsert("tpch.test_date"));
431431
testCases.execute(getQueryRunner(), session, prestoCreateAsSelect(session, "test_date"));
432+
testCases.execute(getQueryRunner(), session, prestoCreateAsSelect(getSession(), "test_date"));
433+
testCases.execute(getQueryRunner(), session, prestoCreateAndInsert(session, "test_date"));
432434
}
433435
}
434436

@@ -547,6 +549,11 @@ private DataSetup prestoCreateAsSelect(Session session, String tableNamePrefix)
547549
return new CreateAsSelectDataSetup(new PrestoSqlExecutor(getQueryRunner(), session), tableNamePrefix);
548550
}
549551

552+
private DataSetup prestoCreateAndInsert(Session session, String tableNamePrefix)
553+
{
554+
return new CreateAndInsertDataSetup(new PrestoSqlExecutor(getQueryRunner(), session), tableNamePrefix);
555+
}
556+
550557
private DataSetup mysqlCreateAndInsert(String tableNamePrefix)
551558
{
552559
return new CreateAndInsertDataSetup(mysqlServer::execute, tableNamePrefix);

presto-phoenix/src/test/java/io/prestosql/plugin/phoenix/TestPhoenixSqlTypeMapping.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -237,6 +237,8 @@ public void testDate()
237237
.setTimeZoneKey(TimeZoneKey.getTimeZoneKey(timeZoneId))
238238
.build();
239239
prestoTestCases.execute(getQueryRunner(), session, prestoCreateAsSelect(session, "test_date"));
240+
prestoTestCases.execute(getQueryRunner(), session, prestoCreateAsSelect(getSession(), "test_date"));
241+
prestoTestCases.execute(getQueryRunner(), session, prestoCreateAndInsert(session, "test_date"));
240242
phoenixTestCases.execute(getQueryRunner(), session, phoenixCreateAndInsert("tpch.test_date"));
241243
}
242244
}
@@ -414,6 +416,11 @@ private DataSetup prestoCreateAsSelect(Session session, String tableNamePrefix)
414416
return new CreateAsSelectDataSetup(new PrestoSqlExecutor(getQueryRunner(), session), tableNamePrefix);
415417
}
416418

419+
private DataSetup prestoCreateAndInsert(Session session, String tableNamePrefix)
420+
{
421+
return new CreateAndInsertDataSetup(new PrestoSqlExecutor(getQueryRunner(), session), tableNamePrefix);
422+
}
423+
417424
private DataSetup phoenixCreateAndInsert(String tableNamePrefix)
418425
{
419426
return new CreateAndInsertDataSetup(new PhoenixSqlExecutor(phoenixServer.getJdbcUrl()), tableNamePrefix);

presto-postgresql/src/test/java/io/prestosql/plugin/postgresql/TestPostgreSqlTypeMapping.java

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -945,6 +945,8 @@ public void testDate()
945945
.build();
946946
testCases.execute(getQueryRunner(), session, postgresCreateAndInsert("tpch.test_date"));
947947
testCases.execute(getQueryRunner(), session, prestoCreateAsSelect(session, "test_date"));
948+
testCases.execute(getQueryRunner(), session, prestoCreateAsSelect(getSession(), "test_date"));
949+
testCases.execute(getQueryRunner(), session, prestoCreateAndInsert(session, "test_date"));
948950
}
949951
}
950952

@@ -997,6 +999,8 @@ public void testTime(boolean insertWithPresto, ZoneId sessionZone)
997999

9981000
if (insertWithPresto) {
9991001
tests.execute(getQueryRunner(), session, prestoCreateAsSelect(session, "test_time"));
1002+
tests.execute(getQueryRunner(), session, prestoCreateAsSelect(getSession(), "test_time"));
1003+
tests.execute(getQueryRunner(), session, prestoCreateAndInsert(session, "test_time"));
10001004
}
10011005
else {
10021006
tests.execute(getQueryRunner(), session, postgresCreateAndInsert("tpch.test_time"));
@@ -1040,6 +1044,8 @@ public void testTimestamp(boolean insertWithPresto, ZoneId sessionZone)
10401044

10411045
if (insertWithPresto) {
10421046
tests.execute(getQueryRunner(), session, prestoCreateAsSelect(session, "test_timestamp"));
1047+
tests.execute(getQueryRunner(), session, prestoCreateAsSelect(getSession(), "test_timestamp"));
1048+
tests.execute(getQueryRunner(), session, prestoCreateAndInsert(session, "test_timestamp"));
10431049
}
10441050
else {
10451051
tests.execute(getQueryRunner(), session, postgresCreateAndInsert("tpch.test_timestamp"));
@@ -1581,6 +1587,11 @@ private DataSetup prestoCreateAsSelect(Session session, String tableNamePrefix)
15811587
return new CreateAsSelectDataSetup(new PrestoSqlExecutor(getQueryRunner(), session), tableNamePrefix);
15821588
}
15831589

1590+
private DataSetup prestoCreateAndInsert(Session session, String tableNamePrefix)
1591+
{
1592+
return new CreateAndInsertDataSetup(new PrestoSqlExecutor(getQueryRunner(), session), tableNamePrefix);
1593+
}
1594+
15841595
private DataSetup postgresCreateAndInsert(String tableNamePrefix)
15851596
{
15861597
return new CreateAndInsertDataSetup(new JdbcSqlExecutor(postgreSqlServer.getJdbcUrl()), tableNamePrefix);

0 commit comments

Comments
 (0)