AlenkaF commented on code in PR #41795:
URL: https://github.com/apache/arrow/pull/41795#discussion_r1626061014
##########
docs/source/format/Columnar.rst:
##########
@@ -1078,8 +1078,95 @@ Arrow columnar format. Each logical type uses one of the
above
physical layouts. Nested logical types may have different physical
layouts depending on the particular realization of the type.
-We do not go into detail about the logical types definitions in this
-document as we consider `Schema.fbs`_ to be authoritative.
+.. glossary::
+
+ Null Type
+ Array where all values are null.
+
+ Boolean Type
+ Array of bit-packed values.
+
+ Int Type
+ Array of signed or unsigned integers with 8, 16, 32 or 64 bit width.
+
+ Floating Point Type
+ Array of floating point numbers with half, single or double precision.
+
+ .. TODO: list other types if we agree to keep the glossary
+
+
+Types and their respective Physical Memory Layout
+-------------------------------------------------
+
++--------------------+----------------------------------+------------------------------+
+| Type | Physical Memory Layout | Additional info
stored |
++====================+==================================+==============================+
+| **Primitive** | |
|
++--------------------+----------------------------------+------------------------------+
+| Null | Null Layout | /
|
++--------------------+----------------------------------+------------------------------+
+| Boolean | Fixed-size Primitive Layout | values bit-packed
|
Review Comment:
Would the table header serve this missing information? Rendered version:
http://crossbow.voltrondata.com/pr_docs/41795/format/Columnar.html#types-and-their-respective-physical-memory-layout
What I could also do is to repeat the table header in between also (before
nested and special)
--
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]
