2010YOUY01 commented on issue #8045: URL: https://github.com/apache/arrow-datafusion/issues/8045#issuecomment-1806701696
> Thanks for driving this forward @2010YOUY01 -- it is very much appreciated. > > I am planning to merge #8079 in 4 more days (after it has been open for a week to ensure anyone who wants has had a chance to review it) > > In the mean time, I was thinking what other work could be done while we are waiting. I see two options: > > 1. We can just start on other tasks as draft PRs that build on [Make fields of `ScalarUDF` , `AggregateUDF` and `WindowUDF` non `pub` #8079](https://github.com/apache/arrow-datafusion/pull/8079) > 2. We could potentially start on the following: > > > Implement some way for `expr_fn` to work using function string names rather than fully resolved `ScalarUDF` as discussed in [#8046 (comment)](https://github.com/apache/arrow-datafusion/pull/8046#issuecomment-1795967063)) > > What do you think? Is any of that interesting to you? I'm going to work on 2 in the next few days. I think this `expr_fn` task should better be done before migrating the first function. -- 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]
