[C++] Arrow metadata not applied recursively when reading Parquet file

[asfimport]

Currently, ApplyOriginalMetadata in src/parquet/arrow/schema.cc is only applied for the top-level node of each schema field. Nested metadata (such as dicts-inside-lists, etc.) will not be applied.

Reporter: Antoine Pitrou / @pitrou
Assignee: Antoine Pitrou / @pitrou

PRs and other links:

• GitHub Pull Request #8366

_{Note: This issue was originally created as ARROW-9943. Please see the migration documentation for further details.}

[asfimport]

Antoine Pitrou / @pitrou:
@emkornfield Are you working on this already? Otherwise I may try to do it before 2.0.0.

[asfimport]

Micah Kornfield / @emkornfield:
Please go ahead and pick this up. I think getting it into 2.0 will be good because it enables things like reading LargeList/FixedSizeList as nested elements.

[asfimport]

Micah Kornfield / @emkornfield:
If you could do it based off of https://github.com/emkornfield/arrow/tree/read_most_types that could save me some merge conflicts. The branch is my work in progress for ARROW-7960 (I need to add tests to verify my code for maps works, but other round trips seems to ework, I had to fix some bugs for FixedSizeList)

[asfimport]

Antoine Pitrou / @pitrou:
Well... ideally we start the release process by the end of the week, so I don't think that's workable.

[asfimport]

Antoine Pitrou / @pitrou:
Issue resolved by pull request 8366
#8366