ggershinsky commented on code in PR #250:
URL: https://github.com/apache/parquet-format/pull/250#discussion_r1620050809
##########
src/main/thrift/parquet.thrift:
##########
@@ -881,15 +922,21 @@ struct ColumnChunk {
/** Crypto metadata of encrypted columns **/
8: optional ColumnCryptoMetaData crypto_metadata
- /** Encrypted column metadata for this chunk **/
+ /** Encrypted column metadata for this chunk
+ *
+ * PAR3: Not set see column_metadata_page on FileMetadata struct
Review Comment:
We can move (copy) the field 8 (`optional ColumnCryptoMetaData
crypto_metadata`) to a V3 per column metadata structure that applies to all row
groups / chunks. The field 9 (`optional binary encrypted_column_metadata`) is
simply the encrypted version of field 3 (`optional ColumnMetaData meta_data`) -
to hide the stats - so should be set per chunk (row group).
--
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]
