[PR] PARQUET-2363: ParquetRewriter should encrypt the V2 page header [parquet-mr]

2023-10-13 Thread via GitHub
ConeyLiu opened a new pull request, #1169: URL: https://github.com/apache/parquet-mr/pull/1169 Make sure you have checked _all_ steps below. This PR adds support for encrypting the V2 page header when encrypting/re-encrypting files. ### Jira - [ ] My PR addresses the fol

Re: [PR] PARQUET-2363: ParquetRewriter should encrypt the V2 page header [parquet-mr]

2023-10-13 Thread via GitHub
ConeyLiu commented on code in PR #1169: URL: https://github.com/apache/parquet-mr/pull/1169#discussion_r1357998012 ## parquet-hadoop/src/main/java/org/apache/parquet/hadoop/ParquetFileWriter.java: ## @@ -784,13 +830,38 @@ public void writeDataPageV2(int rowCount, int nullCount,

Re: [PR] PARQUET-2363: ParquetRewriter should encrypt the V2 page header [parquet-mr]

2023-10-13 Thread via GitHub
ConeyLiu commented on PR #1169: URL: https://github.com/apache/parquet-mr/pull/1169#issuecomment-1761189862 Hi @wgtmac, please help to review this when you are free. Thanks a lot. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] PARQUET-2363: ParquetRewriter should encrypt the V2 page header [parquet-mr]

2023-10-15 Thread via GitHub
wgtmac merged PR #1169: URL: https://github.com/apache/parquet-mr/pull/1169 -- 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: dev-unsubscr...@parquet.apac

Re: [PR] PARQUET-2363: ParquetRewriter should encrypt the V2 page header [parquet-mr]

2023-10-15 Thread via GitHub
ConeyLiu commented on PR #1169: URL: https://github.com/apache/parquet-mr/pull/1169#issuecomment-1763675261 Thanks @wgtmac -- 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.