pitrou commented on code in PR #37526:
URL: https://github.com/apache/arrow/pull/37526#discussion_r1323037564
##########
format/Message.fbs:
##########
@@ -99,6 +99,12 @@ table RecordBatch {
/// Optional compression of the message body
compression: BodyCompression;
+
+ /// Some types such as Utf8View are represented using a variable number of
buffers.
+ /// For each such Field in the pre-ordered flattened logical schema, there
will be
+ /// an entry in variadicBufferCounts to indicate the number of extra buffers
which
+ /// belong to that Field.
Review Comment:
Not sure it's worth spelling it out explicitly, but suggestion below:
```suggestion
/// belong to that Field.
/// If this is omitted, then the data must not contain any Field with a
variable
/// number of buffers, such as BinaryView and Utf8View.
```
--
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]
