[ https://issues.apache.org/jira/browse/ARROW-6820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16967123#comment-16967123 ]
Bryan Cutler commented on ARROW-6820: ------------------------------------- I don't have a strong preference to specific naming, but we should try to be consistent. In C++ it is very confusing because many APIs are "key" and "item" because when MapArray is viewed as a list of structs, the term "value" would mean an element in the struct array. Also, there could be conflicts because "value" is already used in List APIs. I think we should stick with the terminology from Schema.fbs where map type is specified as having a child field "entry", itself with children "key" and "value". In C++ we could work around the API by overriding then renaming, e.g. {code}std::shared_ptr<Array> MapArray::list_values() { return ListArray::values(); }{code} > [C++] [Doc] [Format] Map specification and implementation inconsistent > ---------------------------------------------------------------------- > > Key: ARROW-6820 > URL: https://issues.apache.org/jira/browse/ARROW-6820 > Project: Apache Arrow > Issue Type: Bug > Components: C++, Documentation, Format > Reporter: Antoine Pitrou > Priority: Blocker > Fix For: 1.0.0 > > > In https://arrow.apache.org/docs/format/Layout.html#map-type, the map type is > specified as having a child field "pairs", itself with children "keys" and > "items". > In https://github.com/apache/arrow/blob/master/format/Schema.fbs#L60, the map > type is specified as having a child field "entry", itself with children "key" > and "value". > In the C++ implementation, a map type has a child field "entries", itself > with children "key" and "value". -- This message was sent by Atlassian Jira (v8.3.4#803005)