Skip to content

[SS-69] use updated iceberg-rust fork. new auth hooks for S3 Tables REST and S3 FileIO - #36692

Merged
ublubu merged 3 commits into
mainfrom
kynan/new-iceberg-rust
Jun 10, 2026
Merged

[SS-69] use updated iceberg-rust fork. new auth hooks for S3 Tables REST and S3 FileIO#36692
ublubu merged 3 commits into
mainfrom
kynan/new-iceberg-rust

Conversation

@ublubu

@ublubu ublubu commented May 22, 2026

Copy link
Copy Markdown
Contributor

In preparation for sinking to GCP's Lakehouse, I've pulled in the stable release v0.9.0 for iceberg-rust and added a RequestAuthenticator trait for customizing auth support.

Auth options:

  • AWS SigV4 (now in this PR, previously in our iceberg-rust fork)
  • GCP (via gcp_auth in an upcoming PR)

This PR is still in Draft state because it depends on iceberg-rust changes that have not yet been reviewed. (e.g. MaterializeInc/iceberg-rust#1)

@patrickwwbutler patrickwwbutler 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.

Seems fine to me! I would just make sure you test the changes with a real connection, and not just in testdrive

@ublubu
ublubu marked this pull request as ready for review June 2, 2026 14:12
@ublubu
ublubu requested a review from a team as a code owner June 2, 2026 14:12
Comment thread src/storage-types/src/connections.rs Outdated
Comment thread src/storage-types/src/connections.rs Outdated
@ublubu
ublubu force-pushed the kynan/new-iceberg-rust branch 3 times, most recently from bb82c43 to a318cff Compare June 8, 2026 19:28
@ublubu
ublubu force-pushed the kynan/new-iceberg-rust branch from a318cff to 46500a6 Compare June 10, 2026 14:35
@ublubu
ublubu enabled auto-merge (squash) June 10, 2026 14:40
@ublubu
ublubu merged commit 1310fbc into main Jun 10, 2026
116 checks passed
@ublubu
ublubu deleted the kynan/new-iceberg-rust branch June 10, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants