Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add iceberg.rest-catalog.prefix as a catalog session property #24134

Open
goober opened this issue Nov 14, 2024 · 1 comment
Open

Add iceberg.rest-catalog.prefix as a catalog session property #24134

goober opened this issue Nov 14, 2024 · 1 comment
Labels
iceberg Iceberg connector

Comments

@goober
Copy link

goober commented Nov 14, 2024

Proposal

Enable the iceberg.rest-catalog.prefix to be set as a catalog session property.

Rationale

The Nessie catalog uses the iceberg.rest-catalog.prefix property to point to the catalog branch to use. See Nessie documentation. If it was possible to set this property on a session it would enable a Trino user to independently write and read data on any given branch that is available in the Nessie catalog. Even though the branch creation is handled outside of Trino

If the proposal is accepted I would be happy to contribute.

@goober goober changed the title Add iceberg.rest-catalog.prefix as a session property Add iceberg.rest-catalog.prefix as a catalog session property Nov 14, 2024
@ebyhr ebyhr added the iceberg Iceberg connector label Nov 14, 2024
@ebyhr
Copy link
Member

ebyhr commented Nov 14, 2024

@goober Allowing branch changes with session property may lead to mistake because users have to remember the current session values.

We started the discussion about the branching syntax recently. We will share the document once it's ready for publish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iceberg Iceberg connector
Development

No branches or pull requests

2 participants