juliuszsompolski commented on pull request #28705: URL: https://github.com/apache/spark/pull/28705#issuecomment-638116989
@cloud-fan > Is it possible to always enable java 8 time API in the thrifter server? We could, the cleanest way would be to add the set config to the session config in withLocalProperties in https://github.com/apache/spark/pull/28671. But the session timezone issues there are resolved by that PR already anyway - the Timestamp object collected from the query gets converted to string using HiveResult.toHiveString anyway. No matter if it's a Timestamp, or an Instant, it will result in the same string, working correctly with the session timezone.... ---------------------------------------------------------------- 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
