alamb commented on PR #12654: URL: https://github.com/apache/datafusion/pull/12654#issuecomment-2380273934
> thanks @akurmustafa I feel we can move it to arrow-rs as for me it is natural API evolvement on top of `take` which is used by downstream projects. > > Another thing came to my mind we probably want to optimize `take` as it likely a bottleneck for SMJ we found in [apache/datafusion-comet#901 (comment)](https://github.com/apache/datafusion-comet/issues/901#issuecomment-2332953267) Figuring out how to make `take ` faster would certainly be a good idea, though I think that will require some significant effort. There is not a lot of "low hanging fruit" in that kernel that I know of -- 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]
