divjotarora commented on code in PR #3761: URL: https://github.com/apache/parquet-java/pull/3761#discussion_r3925930203
########## 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: Is this actually a duplicate? This parameter is under section `Class: ParquetOutputFormat` and references `EncryptionPropertiesFactory`. The duplicate below is under section `Class: HadoopReadOptions` and references `DecryptionPropertiesFactory`. I'm fine removing it if there's precedent to not duplicate, but it does seem to be used for two different things. -- 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]
