Skip to content

fix: qualify Snowflake function discovery by database - #842

Merged
kvlonge merged 2 commits into
mainfrom
fix/snowflake-function-catalog
Sep 26, 2026
Merged

kvlonge merged 2 commits into
mainfrom
fix/snowflake-function-catalog

Conversation

@kvlonge

@kvlonge kvlonge commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

Why

SQL tests requiring managed UDFs fail during Snowflake function discovery when the connection has no current database, despite an explicit target database. Discovery can also search the wrong catalog when the session database differs.

Closes CHI-401.

Changes

  • Qualify function metadata with the existing information-schema relation helper.
  • Cover explicit catalogs, identifier escaping and the session-catalog fallback.
  • Add a real-Snowflake regression for discovery without a current database.

Verification

  • Regression tests failed before the fix; 52 focused adapter unit tests pass afterwards.
  • Focused lint, formatting, diff and public-repository hygiene checks pass; make check-ci was not run locally.
  • Scoped independent review found no correctness issues.
  • Real-Snowflake regression is included for credentialed CI; not executed locally.

@kvlonge
kvlonge enabled auto-merge (squash) September 26, 2026 16:27
@kvlonge
kvlonge merged commit 264f84c into main Sep 26, 2026
21 checks passed
@kvlonge
kvlonge deleted the fix/snowflake-function-catalog branch September 26, 2026 16:51
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.

1 participant