Dandandan commented on PR #7422: URL: https://github.com/apache/arrow-rs/pull/7422#issuecomment-2825068017
> Is it fair to say that in general you've done a few refactors recently that replace `MutableBuffer` with `Vec` or collecting directly into the target `Buffer` type? Is there a particular pattern with `MutableBuffer` that we should be avoiding? I think at this point there is little point in using `MutableBuffer` over `Vec` as the latter provides more performant (specialization over `T`, better inlining), slightly more safe and more complete API. The same probably applies for a lot of `Builder`-type APIs probably. It might be worth spending some time documenting the best and most performant way to construct / transform Arrow arrays and apply it accross the arrow-rs create (marking stuff as deprecated if needed). -- 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
