You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2021/11/08 16:16:09 UTC

[GitHub] [hudi] xushiyan commented on pull request #3614: [HUDI-2370] Supports data encryption

xushiyan commented on pull request #3614:
URL: https://github.com/apache/hudi/pull/3614#issuecomment-963317139


   @liujinhui1994 thanks for making this feature. As discussed, some suggestions on the next steps:
   
   We'd like to see this gets into 0.10, but only without major parquet version upgrades. Understand you're using `org.apache.parquet.crypto.*` that only available from 1.12. Would it be possible to keep the feature in a separate module say `hudi-security` and make it useable only when people build Hudi with parquet 1.12? the main question here is whether building with parquet 1.12 makes other features work or not.
   
   Kindly share with us any compatibility issues from your testings. cc @vinothchandar @nsivabalan 
   


-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org