alamb commented on PR #6643: URL: https://github.com/apache/arrow-rs/pull/6643#issuecomment-2561208065
Thank you very much again @jagill for this contribution. Sorry for the delay in reviewing / that this PR is somewhat stuck From my perspective having partial support for a feature (list encoded json structs) is non ideal and will increase the maintenance burden if we are not careful. I think it runs the risk of being a little used / unknown feature. So in order to move this forward I think we need: 1. Additional documentation explaining the feature along with its rationale and an example. Perhaps as an example on https://docs.rs/arrow-json/54.0.0/arrow_json/reader/struct.ReaderBuilder.html -- for rationale you could repurpose some of the comments / dicussion in this PR linked to the trino docs, etc 2. File a ticket (maybe even prepare a PR) to track the work to add writing this version of JSON -- 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]
