MaxGekk commented on issue #25189: [SPARK-28435][SQL] Support accepting the interval keyword in the schema string URL: https://github.com/apache/spark/pull/25189#issuecomment-517372382 > ... if we should revisit the interval type and make it follow SQL standard. cc @MaxGekk @mgaido91 I think we should support 2 separate data types for intervals as it is defined by SQL standard. I opened 2 JIRA tickets for that: [SPARK-27790](https://issues.apache.org/jira/browse/SPARK-27790) ([SPARK-27791](https://issues.apache.org/jira/browse/SPARK-27791) & [SPARK-27793](https://issues.apache.org/jira/browse/SPARK-27793)) but both types must have the same keyword for literals - `interval`. I would keep existing type `CalendarInterval` as is. I don't see any reasons to assign more generic name to it because it is just mix of 2 types with not very well defined semantics.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
