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 support for Create Iceberg Table statement for Snowflake parser #1664

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Vedin
Copy link

@Vedin Vedin commented Jan 17, 2025

Add support for Snowflake CREATE ICEBERG TABLE statement. https://docs.snowflake.com/en/sql-reference/sql/create-iceberg-table.
I decided not to extend the CREATE TABLE statement because it seems to be different and CREATE ICEBERG TABLE only supported in Snowflake.

@Vedin Vedin changed the title Create Iceberg Table Snowflake statement parser Add support for Create Iceberg Table statement for Snowflakeparser Jan 17, 2025
@Vedin Vedin changed the title Add support for Create Iceberg Table statement for Snowflakeparser Add support for Create Iceberg Table statement for Snowflake parser Jan 17, 2025
/// Snowflake-specific statement
/// <https://docs.snowflake.com/en/sql-reference/sql/create-iceberg-table>
/// ```
CreateIcebergTable {
Copy link
Contributor

Choose a reason for hiding this comment

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

The preferred path would be to reuse the CreateTable variant, by e.g. adding a is_iceberg_table: bool to flag the table type and then including any iceberg table fields as optional/empty

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