alamb commented on PR #7597: URL: https://github.com/apache/arrow-rs/pull/7597#issuecomment-2941327496
Ok, I think this PR is ready for a real review and hopefully merge. My proposed next steps are: 1. Merge this PR 2. Iterate in subsequent PRs to improve the kernel using the benchmarks 3. Eventually work up to adding a special `push_filtered` method for skipping the filter step It pains me to leave so much potential performance (I really want to try several of the ones that @Dandandan has listed), but I think we can do them as follow on PRs because: 1. This API should remain stable 2. The kernel is basically "state of the art" (aka I copied it from DataFusion) in terms of performance. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
