emkornfield commented on pull request #7110: URL: https://github.com/apache/arrow/pull/7110#issuecomment-624994025
> Seems pretty straightforward to read for anyone who knows about Arrow's data representation to me. A bit verbose for the simpler cases, however. Maybe we would need to add a couple extra keys at the top level... Versioning for the text schema? Something that says this is an Arrow schema? Or maybe that's not necessary. I would be prefer not to add anything that the flatbuffer parser can't parse. Feature versioning is something I've been meaning to make a proposal for but I think that might be different then what you are thinking of. I'd be OK with having this as a non-official function (probably move out of IPC namespace) to clarify this is only an convenience in the C++ (and binding code base). I think @wesm had some concerns in general about exposing this. ---------------------------------------------------------------- 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]
