jorisvandenbossche commented on code in PR #41257:
URL: https://github.com/apache/arrow/pull/41257#discussion_r1592671363
##########
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``.
Review Comment:
One further nit: "StringArray" is not really generic terminology that we use
in the format specs (it's the name of the array/vector in Arrow C++ and
pyarrow, but not necessarily in other implementations). So I think you can
leave out the `Array` part in each of those three types.
(strictly speaking the spec actually uses Utf8 / LargeUtf8 / Utf8View, but I
think in many places we do use "String" instead)
--
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]
