wjones127 commented on code in PR #37526: URL: https://github.com/apache/arrow/pull/37526#discussion_r1323574646
########## docs/source/format/Columnar.rst: ########## @@ -106,8 +106,10 @@ the different physical layouts defined by Arrow: * **Primitive (fixed-size)**: a sequence of values each having the same byte or bit width * **Variable-size Binary**: a sequence of values each having a variable - byte length. Two variants of this layout are supported using 32-bit - and 64-bit length encoding. + byte length. Three variants of this layout are supported using + * 32-bit offset encoding + * 64-bit offset encoding + * 128-bit view-or-inline encoding Review Comment: As in: a separate bullet from `Variable-size Binary`? If so I'd agree, though if we do we should make sure to differentiate it by saying something like "Unlike the Variable-size binary layout, the elements in this layout may be out-of-order or even overlapping." Without any clarification on the differences, I agree it might be confusing for a reader of the spec. -- 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]
