Re: [I] Consider using `Vec` directly in builders [arrow-rs]

[via GitHub]

alamb commented on issue #7383:
URL: https://github.com/apache/arrow-rs/issues/7383#issuecomment-3107168981

   So specifically, for the PR referenced in 
https://github.com/apache/arrow-rs/pull/7309#discussion_r2023700374
   
   I think it means changing stuff like
   
   
https://github.com/apache/arrow-rs/blob/721150286b00bece40ffcc6f5ac14ebb5d64785b/arrow-array/src/builder/generic_bytes_builder.rs#L33-L32
   
   ```rust
   pub struct GenericByteBuilder<T: ByteArrayType> {
       value_builder: UInt8BufferBuilder,
       offsets_builder: BufferBuilder<T::Offset>,
       null_buffer_builder: NullBufferBuilder,
   }
   ```
   
   To 
   ```rust
   pub struct GenericByteBuilder<T: ByteArrayType> {
       value_builder: Vec<u8>,
       offsets_builder: Vec<T::Offset::Native>,
       null_buffer_builder: NullBufferBuilder,
   }
   ```


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