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

Reply via email to