CurtHagenlocher commented on PR #304: URL: https://github.com/apache/arrow-dotnet/pull/304#issuecomment-4173467636
> IMO it would be nice to update the builders to use the new `NativeBuffer` type, but I can't see how to do that in a backwards compatible way as they expose `ArrowBuffer.Builder` instances as protected members. Yeah, me either. But I think this is a good primitive; I've used it in two different places now but (as you saw) I had to use Reflection to create the `ArrowBuffer`. I was thinking this was also the right primitive to use for some of the cast/convert/operations work that @mobiusklein is doing. > Are we going to end up with two different builders for each array type and a new builder interface? Maybe :(. -- 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]
