Skip to content

refactor!(storage): Move OpenDal Storage to a new crate iceberg-storage-opendal#2207

Merged
blackmwk merged 9 commits intoapache:mainfrom
CTTY:ctty/storage-crate
Mar 5, 2026
Merged

refactor!(storage): Move OpenDal Storage to a new crate iceberg-storage-opendal#2207
blackmwk merged 9 commits intoapache:mainfrom
CTTY:ctty/storage-crate

Conversation

@CTTY
Copy link
Collaborator

@CTTY CTTY commented Mar 4, 2026

Which issue does this PR close?

What changes are included in this PR?

  • Moved OpenDal Storage impl to a new crate iceberg-storage-opendal(crates/storage/opendal)
  • Renamed feature flags storage-s3, storage-xxx to opendal-s3, opendal-xxx

Are these changes tested?

Relying on the existing uts

@CTTY CTTY added the breaking label Mar 4, 2026
@CTTY CTTY marked this pull request as ready for review March 5, 2026 00:34
Copy link
Contributor

@blackmwk blackmwk left a comment

Choose a reason for hiding this comment

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

Thanks @CTTY for this pr, it looks great! Just some minor comment.

Copy link
Contributor

@blackmwk blackmwk left a comment

Choose a reason for hiding this comment

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

Thanks @CTTY for this pr!

@blackmwk blackmwk merged commit c408449 into apache:main Mar 5, 2026
20 checks passed
@CTTY CTTY deleted the ctty/storage-crate branch March 5, 2026 18:33
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.

Move OpenDal Storage impl to a new crate iceberg-storage-opendal

2 participants