Skip to content

feat(snowflake)!: transpilation support MAP_PICK #7189

Merged
geooo109 merged 7 commits intomainfrom
RD-1147671-transpile_map_pick
Mar 5, 2026
Merged

feat(snowflake)!: transpilation support MAP_PICK #7189
geooo109 merged 7 commits intomainfrom
RD-1147671-transpile_map_pick

Conversation

@fivetran-ashashankar
Copy link
Copy Markdown
Collaborator

@fivetran-ashashankar fivetran-ashashankar commented Mar 2, 2026

Adds transpilation support for Snowflake MAP_PICK to duckdb

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 2, 2026

SQLGlot Integration Test Results

Comparing:

  • this branch (sqlglot:RD-1147671-transpile_map_pick, sqlglot version: RD-1147671-transpile_map_pick)
  • baseline (main, sqlglot version: 29.0.2.dev29)

⚠️ Limited to dialects: duckdb

By Dialect

dialect main sqlglot:RD-1147671-transpile_map_pick transitions links
duckdb -> duckdb 4003/4003 passed (100.0%) 4003/4003 passed (100.0%) No change full result / delta

Overall

main: 4003 total, 4003 passed (pass rate: 100.0%), sqlglot version: 29.0.2.dev29

sqlglot:RD-1147671-transpile_map_pick: 4003 total, 4003 passed (pass rate: 100.0%), sqlglot version: RD-1147671-transpile_map_pick

Transitions:
No change

Copy link
Copy Markdown
Collaborator

@georgesittas georgesittas left a comment

Choose a reason for hiding this comment

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

A couple of comments from me.

Comment on lines +2600 to +2602
# Test type inference with array-typed column
from sqlglot.optimizer import qualify
from sqlglot.schema import MappingSchema
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's avoid this in the future (importing into function tests).

@geooo109 geooo109 merged commit 4a3254f into main Mar 5, 2026
9 checks passed
@geooo109 geooo109 deleted the RD-1147671-transpile_map_pick branch March 5, 2026 11:11
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