xushiyan edited a comment 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? say `-Dcustom.build=true`. 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
