burnpanck commented on issue #45880: URL: https://github.com/apache/arrow/issues/45880#issuecomment-2748365244
I would argue that, if `value_parent_indices` isn't to be supported on `MapArray`, it shouldn't be there in the first place. Following your description, it appears that today, it is there through inheritance from `ListArray`, which `MapArray` should not inherit from if it doesn't behave as a list (e.g. of key-value pairs). That said, I think the operation provided by `value_parent_indices` is well-defined and useful for `MapArray`, e.g. to transform a `Map` into something like a `Struct` (or, in pandas speak, to transform from long format to wide format), . It shouldn't matter how many child arrays there are; `value_parent_indices` only ever touches the *offset* array of either `List` or `Map`. In that sense, `Map` is roughly equivalent to a list of structs, and thus indeed is list-like. -- 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