ggershinsky commented on PR #18337:
URL: https://github.com/apache/iceberg/pull/18337#issuecomment-6010465089

   > Open question: local keys use `encryption.data-key-length`, while a KMS
   generated key uses the KMS client's own key spec (e.g. AWS dataKeySpec).
   These could differ; happy to add validation if reviewers want it.
   
   Yes, definitely, I think we need clarity and consistency here for the users. 
I'd support introducing a new parameter ~ `encryption.kek-length`, that is used 
for local kek generation. If the KMS key generation is toggled on, each KMS 
client needs to make sure it uses a spec corresponding to the value of this 
parameter.


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to