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


##########
docs/source/format/CanonicalExtensions.rst:
##########
@@ -251,6 +251,25 @@ 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
+  ``LargeStringArray``or ``StringViewArray``.

Review Comment:
   I think it's ok, but the Large variants can perhaps be omitted since we're 
unlikely to store a JSON value more than 2 GB large.
   
   Similarly, for `arrow.variable_shape_tensor` we're mandating a List type, 
not LargeList.



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