tobixdev commented on PR #8543: URL: https://github.com/apache/arrow-rs/pull/8543#issuecomment-3375477046
> Looks good. This clearly fixes the incorrect behavior, and adding `from_trusted_len_iter` makes this consistent with the primitive arrays. I would prefer if we did not need the intermediate vec allocation, but will write my thoughts on improvements to `BooleanBuilder` in a separate issue. Yes, I agree that it would be great to get similar / better performance without any heap allocations. Thanks for creating the issue! -- 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]
