[GitHub] [arrow] wjones127 commented on pull request #35146: GH-29781: [C++][Parquet] Switch to use compliant nested types by default

["wjones127 (via GitHub)" <gi...@apache.org>]

wjones127 commented on PR #35146:
URL: https://github.com/apache/arrow/pull/35146#issuecomment-1515178867

   > In C++, code shows `force`. But when I go through code in arrow-rs, it doesn't say is force. And though parquet-format use "element" in description, it doesn't says it's forced. Maybe we should ask the parquet maillist later?
   
   The rules were laid out earlier here:
   
   https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#backward-compatibility-rules
   
   Before this PR I did work to make sure that the Arrow C++ implementation (1) didn't care about the field names in equality comparison ([PR](https://github.com/apache/arrow/pull/14847)) and (2) could cheaply cast between types that differed only in field names ([PR](https://github.com/apache/arrow/pull/14198)).


-- 
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-unsubscribe@arrow.apache.org

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