cloud-fan commented on issue #26134: [SPARK-29486][SQL] CalendarInterval should have 3 fields: months, days and microseconds URL: https://github.com/apache/spark/pull/26134#issuecomment-543066743 To be more clear, the timestamp in Spark is `timestamp + zoneId`, and we need extra effort to resolve the zoneId to zoneOffset according to the associated timestamp value. This doesn't follow the SQL standard. We should define the timestamp type as `timestamp + zoneOffset`. That said, the session local timezone should be a zoneOffset. Users should still be able to specify any zoneId as session local timezone, for backward compatibility. We should convert zoneId to zoneOffset at the beginning of a query, according to the current time of query submission.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
