pitrou commented on a change in pull request #11607:
URL: https://github.com/apache/arrow/pull/11607#discussion_r742931213
##########
File path: docs/source/format/Columnar.rst
##########
@@ -622,11 +619,11 @@ use cases:
* A sparse union is more amenable to vectorized expression evaluation in some
use cases.
* Equal-length arrays can be interpreted as a union by only defining the types
array.
-**Example layout: ``SparseUnion<u0: Int32, u1: Float, u2: VarBinary>``**
+**Example layout: ``SparseUnion<i: Int32, f: Float, s: VarBinary>``**
Review comment:
While we are at it, can we use consistent casing for types (here we use
Int32 and Float, in the dense union section above it's lowercased, and the
examples seem a bit of a mixed bag :-))?
--
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]
