pepijnve commented on PR #23211: URL: https://github.com/apache/datafusion/pull/23211#issuecomment-4812655611
@alamb I kept this as draft for now, but while reviewing the changes I noticed the implementations from DataFusion are almost identical to the versions in the current release of `arrow-rs`. Could you run the `string_concat` benchmark? If there is no performance difference, perhaps this can already be merged. -- 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]
