wgtmac commented on code in PR #43995:
URL: https://github.com/apache/arrow/pull/43995#discussion_r1821082820
##########
cpp/src/parquet/arrow/schema.cc:
##########
@@ -681,6 +681,10 @@ Status ListToSchemaField(const GroupNode& group, LevelInfo
current_levels,
// List of primitive type
RETURN_NOT_OK(
NodeToSchemaField(*list_group.field(0), current_levels, ctx, out,
child_field));
+ } else if (list_group.field_count() == 1 &&
list_group.field(0)->is_repeated()) {
Review Comment:
The rule (4) says that `Otherwise, the repeated field's type is the element
type with the repeated field's repetition.`
If I change my special schema to the one below (which does not use `array`
as the element name):
```
optional group a (LIST) {
repeated group b (LIST) {
repeated int32 c;
}
}
```
Then rule 4 applies to the outer list and rule 1 applies to the inner list,
which results in `list<list<int32>>` again.
--
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]
