Skip to content

Handle SupportsWithPrefix in EncryptingFileIO - #14727

Merged
huaxingao merged 3 commits into
apache:mainfrom
tom-s-powell:tom-s-powell/encrypting-file-io-prefix
Dec 10, 2025
Merged

huaxingao merged 3 commits into
apache:mainfrom
tom-s-powell:tom-s-powell/encrypting-file-io-prefix

Conversation

@tom-s-powell

Copy link
Copy Markdown
Contributor

Returns an implementation of EncryptingFileIO that implements SupportsPrefixOperations if the FileIO being wrapped also implements SupportsPrefixOperations.

@github-actions github-actions Bot added the API label Dec 1, 2025
@tom-s-powell
tom-s-powell marked this pull request as ready for review December 1, 2025 14:21
Comment thread api/src/main/java/org/apache/iceberg/encryption/EncryptingFileIO.java Outdated
@huaxingao

Copy link
Copy Markdown
Contributor

cc @ggershinsky

@ggershinsky ggershinsky left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@huaxingao
huaxingao merged commit 3611313 into apache:main Dec 10, 2025
45 checks passed
@huaxingao

Copy link
Copy Markdown
Contributor

Thanks @tom-s-powell for the PR! Thanks @ggershinsky for the review!

talatuyarer pushed a commit to talatuyarer/iceberg that referenced this pull request Apr 1, 2026
* Handle SupportsWithPrefix in EncryptingFileIO.

* rename

* additional tests for delegation

---------

Co-authored-by: Thomas Powell <tpowell@palantir.com>
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.

4 participants