kou commented on code in PR #45024:
URL: https://github.com/apache/arrow/pull/45024#discussion_r1889449451
##########
format/Schema.fbs:
##########
@@ -125,9 +125,9 @@ table FixedSizeList {
/// may be set in the metadata for this field.
///
/// In a field with Map type, the field has a child Struct field, which then
-/// has two children: key type and the second the value type. The names of the
-/// child fields may be respectively "entries", "key", and "value", but this is
-/// not enforced.
+/// has two children: the first of the key type and second of the value type.
Review Comment:
I'm not good at English but we don't need one more `the`?
```suggestion
/// has two children: the first of the key type and the second of the value
type.
```
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
