ggershinsky commented on code in PR #3761: URL: https://github.com/apache/parquet-java/pull/3761#discussion_r3926634840
########## parquet-hadoop/README.md: ########## @@ -288,12 +288,6 @@ true if the reader is using a `DirectByteBufferAllocator` **Description:** Whether to write out page level checksums. **Default value:** `true` ---- - -**Property:** `parquet.crypto.factory.class` -**Description:** Class implementing EncryptionPropertiesFactory. -**Default value:** None. If not set, the file won't be encrypted by a crypto factory. Review Comment: Thanks. Yep, it makes sense to list the same parameter twice, in both output (encrypt) and read (decrypt) sections. I'll update the patch. -- 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]
