feat(snowflake): set QUOTED_IDENTIFIERS_IGNORE_CASE on session#30
Merged
Abhishek Patil (abhishek-pattern) merged 3 commits intoJul 15, 2026
Merged
Conversation
Add ALTER SESSION SET QUOTED_IDENTIFIERS_IGNORE_CASE = TRUE so quoted identifiers are matched case-insensitively in Snowflake connections. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Document that every Snowflake session opened by this package sets QUOTED_IDENTIFIERS_IGNORE_CASE = TRUE, so quoted identifiers resolve case-insensitively regardless of per-call quoting choices. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Copilot started reviewing on behalf of
Abhishek Patil (abhishek-pattern)
July 15, 2026 06:06
View session
There was a problem hiding this comment.
Pull request overview
Sets a Snowflake session parameter on connect so quoted identifiers are resolved case-insensitively, and documents this behavior in the Metaflow helper docs.
Changes:
- Executes
ALTER SESSION SET QUOTED_IDENTIFIERS_IGNORE_CASE = TRUEfor each Snowflake connection created viametaflow/snowflake_connection.py. - Adds “Notes” to relevant docs calling out the always-on session behavior for quoted identifiers.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/ds_platform_utils/metaflow/snowflake_connection.py | Adds an ALTER SESSION call during connection setup. |
| docs/metaflow/query_pandas_from_snowflake.md | Documents the session’s quoted-identifier case-insensitivity behavior. |
| docs/metaflow/publish.md | Documents the session’s quoted-identifier case-insensitivity behavior for publish queries/audits. |
| docs/metaflow/publish_pandas.md | Documents the session’s quoted-identifier case-insensitivity behavior for pandas publishing. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Chinmay (chinmay-dolli-pattern)
approved these changes
Jul 15, 2026
Abhishek Patil (abhishek-pattern)
deleted the
feat/snowflake-quoted-identifiers-ignore-case
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Clickup --> https://pattern.clickup.com/t/2323726/DS-2569
Summary
ALTER SESSION SET QUOTED_IDENTIFIERS_IGNORE_CASE = TRUEon connect, so quoted identifiers resolve case-insensitively instead of failing on case mismatches.Notessection ofpublish,publish_pandas, andquery_pandas_from_snowflakedocs.Test plan
query_pandas_from_snowflake/publishand confirm it resolves without a "does not exist" error.🤖 Generated with Claude Code