Skip to content

Conversation

@fivetran-kwoodbeck fivetran-kwoodbeck changed the title feat(snowflake)!: Type annotation for ENCRYPT and DECRYPT functions feat(snowflake)!: Type annotation for Snowflake ENCRYPT and DECRYPT functions Dec 30, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 30, 2025

SQLGlot Integration Test Results

Comparing:

  • this branch (sqlglot:feature/annotatetype-encryption, sqlglot version: feature/annotatetype-encryption)
  • baseline (main, sqlglot version: 28.5.1.dev49)

⚠️ Limited to dialects: snowflake

By Dialect

dialect main sqlglot:feature/annotatetype-encryption difference links
snowflake -> duckdb 626/1085 passed (57.7%) 626/1085 passed (57.7%) No change full result / delta
snowflake -> snowflake 981/1085 passed (90.4%) 981/1085 passed (90.4%) No change full result / delta

Overall

main: 2170 total, 1607 passed (pass rate: 74.1%), sqlglot version: 28.5.1.dev49

sqlglot:feature/annotatetype-encryption: 2170 total, 1607 passed (pass rate: 74.1%), sqlglot version: feature/annotatetype-encryption

Difference: No change

Copy link
Collaborator

@geooo109 geooo109 left a comment

Choose a reason for hiding this comment

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

Nice work!

Left some comments.

Moreover, these functions exist only in snowflake ?

@fivetran-kwoodbeck
Copy link
Collaborator Author

@VaggelisD For the other dialects, I checked both ENCRYPT and DECRYPT and did not find anything. The ones that support encryption seem to use the name AES_ENCRYPT or similar.

Copy link
Collaborator

@geooo109 geooo109 left a comment

Choose a reason for hiding this comment

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

Nice, thank you!

@geooo109 geooo109 merged commit 9c54329 into main Dec 31, 2025
9 checks passed
@geooo109 geooo109 deleted the feature/annotatetype-encryption branch December 31, 2025 16:48
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