alamb commented on issue #10245: URL: https://github.com/apache/arrow-rs/issues/10245#issuecomment-5416212235
I had claude code find other uses that we could migrate. The list is below (and I added it to the ticket description) - [ ] [`arrow-arith/src/arity.rs`](https://github.com/apache/arrow-rs/blob/main/arrow-arith/src/arity.rs): [`try_binary` nullable path](https://github.com/apache/arrow-rs/blob/0e581556e8ba4ec4fff4d1e0ed11d44730b92767/arrow-arith/src/arity.rs#L285) builds the output values with `BufferBuilder::<O::Native>` (likely covered by #10518) - [ ] [`arrow-cast/src/cast/mod.rs`](https://github.com/apache/arrow-rs/blob/main/arrow-cast/src/cast/mod.rs): [`cast_byte_container`](https://github.com/apache/arrow-rs/blob/0e581556e8ba4ec4fff4d1e0ed11d44730b92767/arrow-cast/src/cast/mod.rs#L2835) converts offsets between offset types with `BufferBuilder::<TO::Offset>` - [ ] [`arrow-ipc/src/writer.rs`](https://github.com/apache/arrow-rs/blob/main/arrow-ipc/src/writer.rs): [`into_zero_offset_run_array`](https://github.com/apache/arrow-rs/blob/0e581556e8ba4ec4fff4d1e0ed11d44730b92767/arrow-ipc/src/writer.rs#L1295) re-encodes sliced run-ends with `BufferBuilder::<R::Native>` - [ ] [`arrow-row/src/variable.rs`](https://github.com/apache/arrow-rs/blob/main/arrow-row/src/variable.rs): [`decode_binary`](https://github.com/apache/arrow-rs/blob/0e581556e8ba4ec4fff4d1e0ed11d44730b92767/arrow-row/src/variable.rs#L287) builds offsets with `BufferBuilder::<I>` - [ ] [`arrow-select/src/zip.rs`](https://github.com/apache/arrow-rs/blob/main/arrow-select/src/zip.rs): [byte-array zip path](https://github.com/apache/arrow-rs/blob/0e581556e8ba4ec4fff4d1e0ed11d44730b92767/arrow-select/src/zip.rs#L562) uses `OffsetBufferBuilder::<T::Offset>` - [ ] [`arrow-avro/src/reader/record.rs`](https://github.com/apache/arrow-rs/blob/main/arrow-avro/src/reader/record.rs): [`Decoder` enum variants](https://github.com/apache/arrow-rs/blob/0e581556e8ba4ec4fff4d1e0ed11d44730b92767/arrow-avro/src/reader/record.rs#L245) (`Binary`/`String`/`StringView`/`Array`/`Map`) hold `OffsetBufferBuilder<i32>` as long-lived decoder state (~15 sites; a larger refactor than the others since the builders are not per-call scratch buffers) -- 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]
