rouault commented on code in PR #41257:
URL: https://github.com/apache/arrow/pull/41257#discussion_r1573896963


##########
docs/source/format/CanonicalExtensions.rst:
##########
@@ -251,6 +251,27 @@ Variable shape tensor
    Values inside each **data** tensor element are stored in 
row-major/C-contiguous
    order according to the corresponding **shape**.
 
+.. _json_extension:
+
+JSON
+====
+
+* Extension name: `arrow.json`.
+
+* The storage type of this extension is ``StringArray`` or
+  or ``LargeStringArray`` or ``StringViewArray``.
+  Only UTF-8 encoded JSON is supported.
+
+* Extension type parameters:
+
+  This type does not have any parameters.
+
+* Description of the serialization:
+
+  Metadata is either an empty string or a JSON string with an empty object.
+  In the future, additional fields may be added, but they are not required
+  to interpret the array.

Review Comment:
   > 1. Parsing options that describe what features were used to generate the 
JSON. E.g. trailing commas allowed, unquoted field names, etc
   
   ouch, I wouldn't call JSON with trailing commas, or unquoted field names 
JSON. Perhaps it would be wise to specify that JSON here is the one of 
https://www.json.org/json-en.html , that is ECMA-404 , otherwise that would 
defeat the purpose of interoperability between implementations



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