akesling commented on code in PR #45024:
URL: https://github.com/apache/arrow/pull/45024#discussion_r1889608062
##########
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:
It works grammatically either way, but the extra "the" isn't strictly
necessary. Definite articles often need not be repeated in lists. I,
personally, prefer the shorter form but defer to maintainers.
See
https://english.stackexchange.com/questions/178767/definite-article-repetition-in-lists
for a spot of context.
--
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
