MaxGekk commented on a change in pull request #26055: [SPARK-29368][SQL][TEST] 
Port interval.sql

 File path: sql/core/src/test/resources/sql-tests/inputs/postgreSQL/interval.sql
 @@ -0,0 +1,330 @@
+-- Portions Copyright (c) 1996-2019, PostgreSQL Global Development Group
+-- SET IntervalStyle to postgres;
+-- check acceptance of "time zone style"
+-- [SPARK-29369] Accept strings without `interval` prefix in casting to 
+-- [SPARK-29370] Interval strings without explicit unit markings
+-- SELECT INTERVAL '01:00' AS `One hour`;
+-- SELECT INTERVAL '+02:00' AS `Two hours`;
+-- SELECT INTERVAL '-08:00' AS `Eight hours`;
+-- SELECT INTERVAL '-1 +02:03' AS `22 hours ago...`;
+-- SELECT INTERVAL '-1 days +02:03' AS `22 hours ago...`;
+-- [SPARK-29371] Support interval field values with fractional parts
+-- SELECT INTERVAL '1.5 weeks' AS `Ten days twelve hours`;
+-- SELECT INTERVAL '1.5 months' AS `One month 15 days`;
+-- SELECT INTERVAL '10 years -11 month -12 days +13:14' AS `9 years...`;
+-- [SPARK-29382] Support the `INTERVAL` type by Parquet datasource
 Review comment:
   This is too much generic, from my point of view. If we use parquet 
everywhere in the ported tests, `interval.sql` shouldn't be exclusion. And here 
we have concrete problem is the **parquet** datasource doesn't support writing 
values of the interval type.
   > -- [SPARK-29382] Support writing `INTERVAL` type to datasource table
   Does it mean that `INTERVAL` should be supported by all builtin datasources?
   I will change the title of the JIRA ticket to unblock this PR.

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.
For queries about this service, please contact Infrastructure at:

With regards,
Apache Git Services

To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to