paleolimbot commented on issue #923:
URL: 
https://github.com/apache/arrow-nanoarrow/issues/923#issuecomment-5300198538

   This approach seems great and I'd be happy to review a PR! (I can probably 
also circle back and do it but it may take a month or two).
   
   > `ArrowIpcDecoderGetMessageMetadata(...)`
   
   The `const char** `out should probably be a `struct ArrowBuffer* out`, since 
this is what the `ArrowMetadataBuilder` will build you. You could also add 
functions to get a key by name or visit all of the keys in place (I believe 
that would avoid a copy which might matter if your metadata is large).


-- 
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]

Reply via email to