You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "xanderbailey (via GitHub)" <gi...@apache.org> on 2026/09/23 07:25:42 UTC

[I] Add AES GCM streaming encryption test cases [iceberg-verification]

xanderbailey opened a new issue, #11:
URL: https://github.com/apache/iceberg-verification/issues/11

   ### Change
   
   New test surface
   
   ### Spec reference
   
   https://iceberg.apache.org/gcm-stream-spec/
   
   ### Description
   
   As we continue to implement the encryption spec across different languages it would be very valuable to have shared test artefacts for streaming encryption. 
   
   Came up here https://github.com/apache/iceberg-python/pull/3969 in a conversation with @mbutrovich 


-- 
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-unsubscribe@iceberg.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Add AES GCM streaming encryption test cases [iceberg-verification]

Posted by "xanderbailey (via GitHub)" <gi...@apache.org>.
xanderbailey commented on issue #11:
URL: https://github.com/apache/iceberg-verification/issues/11#issuecomment-5813056583

   Noting that we will have to contend with https://github.com/apache/iceberg/issues/18219 as we work on this contribution


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Add AES GCM streaming encryption test cases [iceberg-verification]

Posted by "sungwy (via GitHub)" <gi...@apache.org>.
sungwy commented on issue #11:
URL: https://github.com/apache/iceberg-verification/issues/11#issuecomment-5801286347

   Thanks for raising this @xanderbailey - this sounds like a good candidate for the repository. Would you be interested in making this contribution following https://github.com/apache/iceberg-verification/blob/main/CONTRIBUTING.md ?


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Add AES GCM streaming encryption test cases [iceberg-verification]

Posted by "xanderbailey (via GitHub)" <gi...@apache.org>.
xanderbailey commented on issue #11:
URL: https://github.com/apache/iceberg-verification/issues/11#issuecomment-5813286881

   Just started working on this, one question I have since we don't have a lot of precedence here is should I checkin the implementation in Java for how the test cases are generated? Or just checkin the json?


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Add AES GCM streaming encryption test cases [iceberg-verification]

Posted by "sungwy (via GitHub)" <gi...@apache.org>.
sungwy commented on issue #11:
URL: https://github.com/apache/iceberg-verification/issues/11#issuecomment-5835933014

   > Noting that we will have to contend with https://github.com/apache/iceberg/issues/18219 as we work on this contribution
   
   That's a fantastic note @xanderbailey - it'll be awesome to record the resolved and expected fixture here for the contended case.
   
   > Just started working on this, one question I have since we don't have a lot of precedence here is should I checkin the implementation in Java for how the test cases are generated? Or just checkin the json?
   
   I think just checking in the json, and then explaining the provenance (e.g. written by Java x.y.z [see this example](https://github.com/apache/iceberg-verification/pull/6/changes#diff-239da3e15f65f6903bdd55fdf2980f315c180d4a07ff330387af71b87c847f90R22)) with as much detail as you think is needed for these will do.


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Add AES GCM streaming encryption test cases [iceberg-verification]

Posted by "sungwy (via GitHub)" <gi...@apache.org>.
sungwy commented on issue #11:
URL: https://github.com/apache/iceberg-verification/issues/11#issuecomment-5802576584

   Awesome. Assigned this to you @xanderbailey 
   
   I think this can stay at the top level, parallel to `table-spec` which is proposed in this PR: https://github.com/apache/iceberg-verification/pull/3/changes


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [I] Add AES GCM streaming encryption test cases [iceberg-verification]

Posted by "xanderbailey (via GitHub)" <gi...@apache.org>.
xanderbailey commented on issue #11:
URL: https://github.com/apache/iceberg-verification/issues/11#issuecomment-5801424668

   100% happy to add this 


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org