rdblue commented on code in PR #9359:
URL: https://github.com/apache/iceberg/pull/9359#discussion_r1439979355
##########
core/src/main/java/org/apache/iceberg/encryption/StandardEncryptionManager.java:
##########
@@ -41,7 +42,10 @@ public class StandardEncryptionManager implements
EncryptionManager {
* @param kmsClient Client of KMS used to wrap/unwrap keys in envelope
encryption
*/
public StandardEncryptionManager(
- String tableKeyId, int dataKeyLength, KeyManagementClient kmsClient) {
+ String tableKeyId,
+ int dataKeyLength,
+ KeyManagementClient kmsClient,
+ boolean nativeDataEncryption) {
Review Comment:
I don't think that this should be passed in. The encryption manager needs to
support files that use both native encryption (Parquet) and files that use AES
GCM streams (Avro). There is no way to set this correctly because the behavior
depends on the file type.
Instead, the caller should decide whether to call `decrypt` based on whether
it is needed.
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
