adamreeve commented on issue #341: URL: https://github.com/apache/arrow-dotnet/issues/341#issuecomment-4348412111
Given we vendor the Flatbuffers code, we could modify the builder to work with an `IBufferWriter`. But other projects could probably benefit from this too, and diverging from upstream Flatbuffers too much could cause problems for us later, so it would be nice to get this added to the upstream Flatbuffers project. I think it might be possible to do this in a backwards compatible way by refactoring `FlatBufferBuilder` to work with an `IBufferWriter`, and `ByteBuffer` could implement `IBufferWriter`, or a wrapper class added to implement it. I think the next step would be to raise this as an issue in the Flatbuffers project and see what their thoughts are. There's probably not much point in adding explicit support for `IBufferWriter` without Flatbuffer integration as handling of most other writes like Arrow buffers doesn't require an intermediate buffer. -- 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]
