Skip to content

Support Snowflake-Managed Iceberg Tables via SnowflakeCatalog #1834

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.

Sign up for GitHub

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

Open
srilman wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Support Snowflake-Managed Iceberg Tables via SnowflakeCatalog #1834

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8e76947
Copied from PR and Bodo
srilman Mar 23, 2025
43ac041
Address comment
srilman Mar 23, 2025
15eb260
Address StaticTable comment
srilman Mar 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Address comment
  • Loading branch information
@srilman
srilman committed Mar 23, 2025
commit 43ac041ffd75a4e872c23e85a4a352a7a402e9f0
3 changes: 0 additions & 3 deletions pyiceberg/catalog/snowflake.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -270,11 +270,8 @@ def drop_namespace(self, namespace: str | Identifier) -> None:
)

sf_query = "DROP SCHEMA IF EXISTS (%s)"
db_query = "DROP DATABASE IF EXISTS (%s)"

with self.connection.cursor(DictCursor) as cursor:
if sf_identifier.database:
cursor.execute(db_query, (sf_identifier.database,))
cursor.execute(sf_query, (sf_identifier.schema_name,))

def list_tables(self, namespace: str | Identifier) -> list[Identifier]:
Expand Down