alkis commented on code in PR #542:
URL: https://github.com/apache/parquet-format/pull/542#discussion_r2610047452
##########
src/main/thrift/parquet.thrift:
##########
@@ -958,6 +958,22 @@ union ColumnCryptoMetaData {
struct ColumnChunk {
/** File where column data is stored. If not set, assumed to be same file as
* metadata. This path is relative to the current file.
+ *
+ * As of December 2025, there are no known released Parquet implementations
+ * that make use of this field when reading columns. Readers should check
+ * that the field is empty before retrieving a column from within the file,
+ * and error if it isn't.
+ *
+ * Writers should not populate this field.
+ *
+ * Any new use of this field must go through the normal Parquet feature
+ * addition process. CONTRIBUTING.md in the parquet-format repository
+ * provides details on the process.
Review Comment:
nit: I wouldn't mention CONTRIBUTING.md here - seems something that can be
found easily without explicit mention.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
