wgtmac commented on code in PR #41187: URL: https://github.com/apache/arrow/pull/41187#discussion_r1564762713
########## docs/source/cpp/parquet.rst: ########## @@ -571,6 +571,19 @@ More specifically, Parquet C++ supports: * EncryptionWithFooterKey and EncryptionWithColumnKey modes. * Encrypted Footer and Plaintext Footer modes. +Field Id +---------- + +The parquet format supports an optional integer field_id which can be assigned +to a field. This is used in the `iceberg specification <https://github.com/apache/iceberg/blob/main/format/spec.md#column-projection>` __ Review Comment: I'd suggest not mentioning iceberg to be neutral. ########## docs/source/cpp/parquet.rst: ########## @@ -571,6 +571,19 @@ More specifically, Parquet C++ supports: * EncryptionWithFooterKey and EncryptionWithColumnKey modes. * Encrypted Footer and Plaintext Footer modes. +Field Id Review Comment: It seems that `Roundtripping Arrow types` above is a better place for field-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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
