jagill commented on code in PR #6643: URL: https://github.com/apache/arrow-rs/pull/6643#discussion_r1890535752
########## arrow-json/src/reader/mod.rs: ########## @@ -170,12 +170,30 @@ mod struct_array; mod tape; mod timestamp_array; +/// Specifies what is considered valid JSON when parsing StructArrays. Review Comment: Sadly, the documentation of the format of the returned data is a little lacking. They describe that the top-level row is a list with one entry per column (https://prestodb.io/docs/current/develop/client-protocol.html#important-queryresults-attributes, https://trino.io/docs/current/develop/client-protocol.html#important-queryresults-attributes), but do not specify how any structural elements are serialized (or non-structural things like Datetimes etc). If it helps, I could add this to the Presto documentation. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
