rdblue commented on pull request #1825: URL: https://github.com/apache/iceberg/pull/1825#issuecomment-740213550
> Currently it treats everything as UTC, to be honest I didn't know about spark.sql.session.timeZone. Want me to change it to respect "spark.sql.session.timeZone"? I am personally biased against doing anything with data outside of UTC but if other people are using that setting we should probably not surprise them. Since these can be embedded in SQL queries, I think it makes the most sense to match what Spark does. That should be parsing timestamps without an offset or zone using the SQL session time zone. That may default using `spark.sql.session.timeZone`, but there is usually a way to set it in a session so if we can get a live value from Spark then we should try to do that. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
