beliefer commented on pull request #33588: URL: https://github.com/apache/spark/pull/33588#issuecomment-891676094
ORC includes two different forms of timestamps from the SQL world: ``` Timestamp is a date and time without a time zone, which does not change based on the time zone of the reader. Timestamp with local time zone is a fixed instant in time, which does change based on the time zone of the reader. ``` But The current implementation of spark will treat A TIMESTAMP => TIMESTAMP_LTZ incorrectly. Should we change the behavior, so it will be converted as follows: ``` A TIMESTAMP => TIMESTAMP_NTZ Timestamp with local time zone => TIMESTAMP_LTZ ``` -- 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. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org 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