jackye1995 edited a comment on pull request #2638: URL: https://github.com/apache/iceberg/pull/2638#issuecomment-849001743
The biggest question I have regarding this is that: should we have this concept of "native" encryption? My original thinking is that we should use native encryption as long as the file format supports it and there are enough information provided in key_metadata. For example, a user can specify all those native configurations and enable it, but use Avro that does not have column encryption, then it still could not be applied. The only use case for not using native I think is for backwards compatibility with some other dependent systems. In that case, we only need a boolean config flag for the reader and writer, and I feel it is unnecessary to dedicate specific parameters only for native encryption. -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
