Skip to content

chore: ignore partition_by config if dialect is snowflake#5522

Merged
toriwei merged 5 commits intomainfrom
tori/ignore_partitionby_snowflake
Oct 14, 2025
Merged

chore: ignore partition_by config if dialect is snowflake#5522
toriwei merged 5 commits intomainfrom
tori/ignore_partitionby_snowflake

Conversation

@toriwei
Copy link
Contributor

@toriwei toriwei commented Oct 9, 2025

snowflake does not support partitioning. log a warning and ignore if the config set.

@toriwei toriwei force-pushed the tori/ignore_partitionby_snowflake branch from b1468c9 to 8bb1f16 Compare October 9, 2025 21:51
@toriwei toriwei requested a review from eakmanrq October 9, 2025 23:00
@toriwei toriwei force-pushed the tori/ignore_partitionby_snowflake branch from 4d60f69 to 5fbde41 Compare October 10, 2025 15:32
@toriwei toriwei enabled auto-merge (squash) October 10, 2025 16:24
@toriwei toriwei disabled auto-merge October 10, 2025 16:24
@toriwei toriwei enabled auto-merge (squash) October 10, 2025 16:24
@toriwei toriwei force-pushed the tori/ignore_partitionby_snowflake branch from 5fbde41 to ebcbccd Compare October 10, 2025 16:29
@toriwei toriwei disabled auto-merge October 10, 2025 17:23
@toriwei toriwei force-pushed the tori/ignore_partitionby_snowflake branch from ebcbccd to d941fbe Compare October 10, 2025 20:11
@toriwei toriwei force-pushed the tori/ignore_partitionby_snowflake branch from 80344ea to f73d245 Compare October 13, 2025 16:08
@toriwei toriwei merged commit c62cac1 into main Oct 14, 2025
36 checks passed
@toriwei toriwei deleted the tori/ignore_partitionby_snowflake branch October 14, 2025 18:09
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