Skip to content

Parquet: Add round-trip test for encrypted shredded variant columns - #17469

Open
nssalian wants to merge 3 commits into
apache:mainfrom
nssalian:encryption-variant-shredding-test
Open

Parquet: Add round-trip test for encrypted shredded variant columns#17469
nssalian wants to merge 3 commits into
apache:mainfrom
nssalian:encryption-variant-shredding-test

Conversation

@nssalian

@nssalian nssalian commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

Rationale

Existing tests cover shredding and encryption separately, not together. Adding this to clarify the encryption behavior on a shredded variant.

Changes

  • Adds a test covering a shredded variant column written to an encrypted Parquet file: no-key read fails, keyed read round-trips, and the decrypted footer confirms the shredded layout. Test-only.
  • Cleaned up some of the common artifacts into helpers from the previous tests so we don't have to duplicate.

@nssalian
nssalian marked this pull request as ready for review August 2, 2026 02:39
@nssalian
nssalian requested a review from huaxingao August 2, 2026 02:40
Comment thread parquet/src/test/java/org/apache/iceberg/parquet/TestParquetDataWriter.java Outdated
Comment thread parquet/src/test/java/org/apache/iceberg/parquet/TestParquetDataWriter.java Outdated
Comment thread parquet/src/test/java/org/apache/iceberg/parquet/TestParquetDataWriter.java Outdated
Comment thread parquet/src/test/java/org/apache/iceberg/parquet/TestParquetDataWriter.java Outdated
Comment thread parquet/src/test/java/org/apache/iceberg/parquet/TestParquetDataWriter.java Outdated
@nssalian

nssalian commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator Author

Thanks for the reviews @wombatu-kun. I cleaned up these tests and some of the older ones in the file too so we reuse with helpers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants