[GitHub] [arrow] kiszk commented on a change in pull request #8974: ARROW-10941: Document supported Parquet encryption features

[GitBox <gi...@apache.org>]

kiszk commented on a change in pull request #8974:
URL: https://github.com/apache/arrow/pull/8974#discussion_r548955381



##########
File path: docs/source/cpp/parquet.rst
##########
@@ -244,7 +244,22 @@ Limitations
 
 Writing or reading back FixedSizedList data with null entries is not supported.
 
-.. TODO: document supported encryption features
+Encryption
+----------
+
+Parquet C++ implements all features, specified in the encryption `specification <https://github.com/apache/parquet-format/blob/master/Encryption.md>`__,
+except for encryption of column index and bloom filter modules. 
+
+More specifically, Parquet C++ supports
+
+* AES_GCM_V1 and AES_GCM_CTR_V1 encryption algorithms
+
+* AAD suffix for Footer, ColumnMetaData, Data Page, Dictionary Page, Data PageHeader, Dictionary PageHeader module types. ColumnIndex, OffsetIndex, BloomFilter Header, BloomFilter Bitset are not supported.

Review comment:
       nit: types. -> types,  ?




----------------------------------------------------------------
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.

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