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


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

Review Comment:
   I think the one thing to consider here as possible future extensions (this 
might have already been discussed on ML a while ago), is non-standard encodings 
using Binary types instead of String.
   
   In the short term we should probably recommend that implementations should 
likely reject Binary annotated JSON fields



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