[GitHub] [parquet-mr] shangxinli commented on pull request #958: PARQUET-2138: Add ShowBloomFilterCommand to parquet-cli
shangxinli commented on PR #958: URL: https://github.com/apache/parquet-mr/pull/958#issuecomment-1172927105 Let's merge it now and we can add column decryption later. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [parquet-mr] shangxinli commented on pull request #958: PARQUET-2138: Add ShowBloomFilterCommand to parquet-cli
shangxinli commented on PR #958: URL: https://github.com/apache/parquet-mr/pull/958#issuecomment-1159347960 @WangGuangxin Do you still plan to implement the decryption? I don't want to place a blocker if you don't have plan for it. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [parquet-mr] shangxinli commented on pull request #958: PARQUET-2138: Add ShowBloomFilterCommand to parquet-cli
shangxinli commented on PR #958: URL: https://github.com/apache/parquet-mr/pull/958#issuecomment-1127058252 Yeah, I guess you need to pass the key directly or KMS URL. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [parquet-mr] shangxinli commented on pull request #958: PARQUET-2138: Add ShowBloomFilterCommand to parquet-cli
shangxinli commented on PR #958: URL: https://github.com/apache/parquet-mr/pull/958#issuecomment-1104439140 For column encrypted files, does it work? Can you add tests for it? For how to create encrypted test file, you can refer to this file parquet-hadoop/src/test/java/org/apache/parquet/hadoop/util/ColumnEncryptorTest.java. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org