[PARQUET-1798] [C++] Review logic around automatic assignment of field_id's

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: cpp-5.0.0
• Component/s: parquet-cpp
• Labels: pull-request-available
• Assignee: Weston Pace
• Reporter: Wes McKinney
• Created: 14/Feb/20 16:53
• Updated: 27/May/21 15:56
• Resolved: 27/May/21 15:55

Description

At schema deserialization (from Thrift) time, we are assigning a default field_id to the Schema node based on a depth-first ordering of notes. This means that a round trip (load, then write) will cause field_id's to be written that weren't there before. I'm not sure this is the desired behavior.

We should examine this in more detail and possible change it. See also discussion in ARROW-7080 https://github.com/apache/arrow/pull/6408

Issue Links

• links to: GitHub Pull Request #10289