pitrou commented on code in PR #41795: URL: https://github.com/apache/arrow/pull/41795#discussion_r1626007871
########## 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 | Review Comment: I'm not sure what "additional info" alludes to? This seems to be mostly type parameters, but not only. Mixing concepts together like this feels confusing to me. -- 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]
