Omega359 commented on issue #12218: URL: https://github.com/apache/datafusion/issues/12218#issuecomment-2441726245
I looked into that @alamb but I think what is required is access to ConfigOptions (so we can get tz from config, etc). I don't think `SimplifyInfo` provides that. I was toying with some approaches this weekend for this and didn't come up with anything simple. The most obvious option to me is to add an `InvokeInfo` that provides access to `ExecutionProps` and `ConfigOptions` to the arguments for invoke_batch, etc. That of course is a breaking change though. Beyond that I toyed with: - making UDF's not static which looked like a fairly large amount of work just for this - forcing users that want UDF's to have access to DF config/etc to be registered separately so that they could inject the config 'manually' - using just the system tz for date related things ... which to me might cause more issues than it would solve Augmenting `SimplyInfo` to have `ConfigOptions` could work but it's a lot of effort to rewrite calls just to apply tz info (or anything else that a UDF might want from config) . Seems like applying a jackhammer to insert a screw. -- 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]
