Dandandan commented on pull request #9338: URL: https://github.com/apache/arrow/pull/9338#issuecomment-771441508
@alamb I think you are right, there are two places now `pub fn sort` and `pub fn aggregate` and that do more cloning than before. I could imagine this would lead to a bit more cloning in some cases, although most cases will be passing either a simple constant slice or a slice from an existing vec. I will have a look to your PR. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
