jorisvandenbossche commented on code in PR #33925:
URL: https://github.com/apache/arrow/pull/33925#discussion_r1113944390
##########
docs/source/format/CanonicalExtensions.rst:
##########
@@ -72,4 +72,67 @@ same rules as laid out above, and provide backwards
compatibility guarantees.
Official List
=============
-No canonical extension types have been standardized yet.
+Fixed shape tensor
+==================
+
+* Extension name: `arrow.fixed_shape_tensor`.
+
+* The storage type of the extension: ``FixedSizeList`` where:
+
+ * **value_type** is the data type of individual tensor elements
+ and is an instance of Arrow ``DataType`` or ``Field``.
Review Comment:
```suggestion
* **value_type** is the data type of individual tensor elements.
```
I think saying just this is sufficient (even "DataType" is strictly speaking
a Arrow C++ specific term, and not necessarily each implementation uses that
term). It should also be clear that FixedSizeList is an existing arrow type,
and thus it follows from FixedSizeList definition how its child array is
defined by a field with a type.
--
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]
