pitrou commented on code in PR #37526:
URL: https://github.com/apache/arrow/pull/37526#discussion_r1323032277
##########
docs/source/format/Columnar.rst:
##########
@@ -880,19 +929,20 @@ For the avoidance of ambiguity, we provide listing the
order and type
of memory buffers for each layout.
.. csv-table:: Buffer Layouts
- :header: "Layout Type", "Buffer 0", "Buffer 1", "Buffer 2"
- :widths: 30, 20, 20, 20
-
- "Primitive",validity,data,
- "Variable Binary",validity,offsets,data
- "List",validity,offsets,
- "Fixed-size List",validity,,
- "Struct",validity,,
- "Sparse Union",type ids,,
- "Dense Union",type ids,offsets,
- "Null",,,
- "Dictionary-encoded",validity,data (indices),
- "Run-end encoded",,,
+ :header: "Layout Type", "Buffer 0", "Buffer 1", "Buffer 2", "Variadic
Buffers"
+ :widths: 30, 20, 20, 20, 20
+
+ "Primitive",validity,data,,
+ "Variable Binary",validity,offsets,data,
+ "Variable Binary - View",validity,packed headers,,data
Review Comment:
Nit, but I don't think the hyphen or dash is useful here?
Also, we use the term "view" rather than "headers" above and we don't
mention packing.
```suggestion
"Variable Binary View",validity,views,,data
```
--
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]
