alamb commented on PR #6808: URL: https://github.com/apache/arrow-rs/pull/6808#issuecomment-2599289085
> I believe the penalty introduced by this PR would be mitigated for interleave's case if we also merge https://github.com/apache/arrow-rs/pull/6779, for other cases it feels like the read / transfer over the wire improvements might outweigh the cost. Happy to hear your thoughts I have merged https://github.com/apache/arrow-rs/pull/6779 now I think one of the potential performance concerns is that `MutableBuffer::extend` is used not just for `interleave` but many other operations too (such as `filter` I think) where the improvements in https://github.com/apache/arrow-rs/pull/6779 will not help I don't think -- 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]
