felipecrv commented on PR #41373: URL: https://github.com/apache/arrow/pull/41373#issuecomment-2096599183
> I'm skeptical that you want to reuse this for Filter, unless you add Gather methods for batch selection. For Filter performance, it is essential to write out ranges of selected values at a time, not one value at a time. I don't know if that's what you have in mind. I want to expand the set of `WriteValue` implementations to support writing multiple values. Then add a version of `Gather::Execute*()` that takes boolean arrays (masks) instead of indices (selection vectors). -- 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]
