Skip to content

fix auth tracking for fast-append flows#1169

Merged
bbalser merged 1 commit into
mainfrom
bbalser/fast-append-auth
Apr 1, 2026
Merged

fix auth tracking for fast-append flows#1169
bbalser merged 1 commit into
mainfrom
bbalser/fast-append-auth

Conversation

@bbalser
Copy link
Copy Markdown
Collaborator

@bbalser bbalser commented Apr 1, 2026

  1. catalog/mod.rs:69 — with_auth changed from private to pub(crate)
  2. iceberg_table.rs:62-79 — commit_files now wraps the transaction build + commit inside catalog.with_auth(),
    so a 401 triggers token invalidation and a single retry with a fresh transaction

@bbalser bbalser merged commit 4be70f8 into main Apr 1, 2026
29 checks passed
@bbalser bbalser deleted the bbalser/fast-append-auth branch April 1, 2026 21:13
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.

2 participants