pitrou commented on code in PR #41187: URL: https://github.com/apache/arrow/pull/41187#discussion_r1569082708
########## 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 find it useful to give context, so perhaps the wording can be improved to make it clear that it's just an example? ########## 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 find it useful to give context, so perhaps the wording can be improved to make it clear that it's just an example, rather than remove it entirely? -- 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]
