brancz commented on PR #7671: URL: https://github.com/apache/arrow-rs/pull/7671#issuecomment-2983445430
It appears that the take kernel was already implemented separately (it's probably worth thinking about switching the implementation over to using extend though), but the interleave kernel is implemented using extend, so I'll add tests that use that. Looking at the take kernel implementation I think there is an opportunity for optimizing when encountering REE because it adds indexes one by one, when logical indexes could very well be consecutive which would take advantage. I would suggest let's open an issue for that as a follow up. -- 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]
