PHILO-HE commented on PR #2823: URL: https://github.com/apache/incubator-gluten/pull/2823#issuecomment-2093055772
> > The directly setting for `spark.sql.session.timeZone` in gluten can cause some UT failure because some spark UTs do the tests by setting default timezone, instead of setting this spark config. So in the latest code, we introduced a separate gluten config key to hold default session timezone, which will be used if that spark config is not set by user (this handling is quite consistent with spark's). > > Do you mean spark also have a default timezone? @FelixYBW, yes, Spark also has a default timezone. And it's the OS timezone detected. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
