ritchie46 commented on pull request #269: URL: https://github.com/apache/arrow-rs/pull/269#issuecomment-835750524
I will close this PR and agree that we should continue with `MutableBuffer`. I have still one question; You mention: > Some APIs to build arrays also have untyped Vec<MutableBuffer>, and thus MutableBuffer must also be untyped. Would it make sense to still explore a `MutableBuffer<T> where T: NativeType` that implements a trait `dyn MutableBuffer` for the untyped API. Then we can use the trait objects where the type needs to be "forgotten" I think we can set the generic methods over `T` under a condition `where Self: sized` to make them object safe and add a `bytes` version for when we deal with the trait object instead of the concrete type. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
