[GitHub] [arrow] pitrou commented on pull request #10289: PARQUET-1798: [C++] Review logic around automatic assignment of field_id's

[GitBox <gi...@apache.org>]

pitrou commented on pull request #10289:
URL: https://github.com/apache/arrow/pull/10289#issuecomment-839631978


   I'm not even sure what field_ids are supposed to be for. The parquet spec only has this to say:
   ```thrift
     /** When the original schema supports field ids, this will save the
      * original field id in the parquet schema
      */
     9: optional i32 field_id;
   ```
   
   I suppose "original schema" means something non-Parquet, but what? Is it just some kind of arbitrary application-defined id?


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org