pitrou commented on PR #14369: URL: https://github.com/apache/arrow/pull/14369#issuecomment-1292126171
Thanks @Jedi18 for attempting this. At a high level, two things: 1) Can you first tackle just the C++ side and write tests on that side? 2) > Follow up PRs will update the Table, RecordBatch etc sorting implementations to use this NestedValuesComparator. Did you actually measure performance? I expect this to be slower that the current RecordBatch and Table sorting. -- 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]
