yaooqinn commented on pull request #29064: URL: https://github.com/apache/spark/pull/29064#issuecomment-657346567
the test case for listing all timezones is removed because it varies from different JDKs. The interval is supported and also some extensions were made too 1) offset range is [-18, +18] in spark which is bigger than ansi's, 2) the second part is supported in Spark, e.g. +14:14:14, while ansi only indicates hours-minutes, 3) multi value-units interval is much common than unit-to-unit ones, so this interval form is supported too. cc @cloud-fan ---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org