paleolimbot opened a new pull request, #209: URL: https://github.com/apache/arrow-nanoarrow/pull/209
Currently the decoder always assembles an `ArrowArray` and subsequently validates that array. That caused a problem when validating a union because the `ArrowArray` builder doesn't have a place to put custom type IDs. Also, this is every so slightly wasteful: the `ArrowArray` validation allocates an unnecessary `ArrowArrayView` and validates that. We already have an `ArrowArrayView` to help with the buffer shuffling, so this PR just swaps the direction: the first step is to create the `ArrowArrayView`; the second step is to populate the `ArrowArray` (if requested). TODO: Finish the refactoring, actually try validating unions -- 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]
