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


   @liujinhui1994 per our discussions, some points here
   - `hoodie.parquet.encryption.keys` can be used as the switch. default to not 
set. if not set, then encryption feature is disabled. any parquet settings 
users provide will be ignored
   - If users enable encryption but `hoodie.table.base.file.format` is not 
parquet, there should be some warning to highlight it to the users
   - it'll be good to mark this as experimental feature. you can clarify this 
with more details (like why experimental) in the config's docs and related java 
classes' docs. Release notes will also highlight 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to