alamb commented on PR #13491: URL: https://github.com/apache/datafusion/pull/13491#issuecomment-2496257949
Here is what I suggest we do with this PR: 1. Remove the change for https://github.com/apache/datafusion/pull/13507 (let's do that in a separate PR) 2. Fix up the regressions introduced (where batch_size --> 1) Then merge this one in. If you have more time @joseph-isaacs it would be even better to change this PR so all the functions used `ScalarFunctionArgs` but I realize that is asking quite a bit -- 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]
