alamb opened a new issue, #10245:
URL: https://github.com/apache/arrow-rs/issues/10245

   ### Is your feature request related to a problem or challenge?
   
   As @Jefffrey  has observed in 
https://github.com/apache/arrow-rs/pull/10229#discussion_r3496012625, switching 
from `OffsetBufferBuilder` / `BufferBuilder` to use `Vec<..>` often results in 
a non trivial speedup.
   
   This is likely because the Rust language has a *highly* optimized Vec 
implementation
   
   
   
   
   ### Describe the solution you'd like
   
   Basically try and change any code like this
   
   ```rust
       let mut values = BufferBuilder::<T::Native>::new(len);
   ```
   
   To use 
   ```rust
   let mut values = Vec<T::Native>::with_capacity(len);
   ```
   
   And various equivalent
   
   ## Note to contributors: 
   please make targeted PRs (that change one or two usages each) to keep the 
PRs easy to review and thus fast to merge
   
   
   
   ### Describe alternatives you've considered
   
   _No response_
   
   ### Additional context
   
   _No response_


-- 
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]

Reply via email to