[
https://issues.apache.org/jira/browse/PARQUET-1884?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17167035#comment-17167035
]
ASF GitHub Bot commented on PARQUET-1884:
-----------------------------------------
gszadovszky commented on a change in pull request #800:
URL: https://github.com/apache/parquet-mr/pull/800#discussion_r461553080
##########
File path: parquet-hadoop/README.md
##########
@@ -239,6 +239,12 @@ conf.set("parquet.bloom.filter.expected.ndv#column.path",
200)
**Description:** Whether to write out page level checksums.
**Default value:** `true`
+---
+
+**Property:** `parquet.crypto.factory.class`
+**Description:** Class implementing EncryptionPropertiesFactory.
Review comment:
Please, add double spaces to the end of the lines to enforce newlines.
(Maybe it was my mistake during the rebase that they were removed.)
(You may check the result by reviewing the rendering of the file on the
github UI.)
##########
File path: .travis.yml
##########
@@ -1,4 +1,5 @@
language: java
+jdk: openjdk8
Review comment:
jdk is already listed, this one is not required
----------------------------------------------------------------
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:
[email protected]
> Merge encryption branch into master
> -----------------------------------
>
> Key: PARQUET-1884
> URL: https://issues.apache.org/jira/browse/PARQUET-1884
> Project: Parquet
> Issue Type: Sub-task
> Components: parquet-mr
> Reporter: Gidon Gershinsky
> Assignee: Gidon Gershinsky
> Priority: Major
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
