Weijun-H commented on PR #18017:
URL: https://github.com/apache/datafusion/pull/18017#issuecomment-3407847057
> This is looking good. I'd like to see an addition to the upgrade guide as
this currently will be either a slight change in behaviour (timezone of None
previously vs now it'll be Some("+00:00") with default config) or with a
different tz set in the config it'll result in a different value.
>
> We may want to update the doc for ExecutionOptions::time_zone slightly as
well as currently it's very targetted at 'Extract'.
>
> As a followup PR I think we could allow for when no tz is set in the
config_options to use `None` vs `Some('')`.
tracked by #18081
--
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]
