Skip to content
This repository was archived by the owner on Oct 1, 2026. It is now read-only.

Zdenko/snowflake parse trim with second param - #1

Closed
zdenal wants to merge 4 commits into
mainfrom
zdenko/snowflake-parse-trim-with-second-param
Closed

Zdenko/snowflake parse trim with second param#1
zdenal wants to merge 4 commits into
mainfrom
zdenko/snowflake-parse-trim-with-second-param

Conversation

@zdenal

@zdenal zdenal commented Sep 21, 2023

Copy link
Copy Markdown

Why

We are getting errors with parsing snowflake TRIM expression including removing characters separated by comma.

https://docs.snowflake.com/en/sql-reference/functions/trim

@zdenal zdenal self-assigned this Sep 21, 2023
@zdenal

zdenal commented Sep 22, 2023

Copy link
Copy Markdown
Author

Close this one in favour #2 ... without rust formatter changes.

@zdenal zdenal closed this Sep 22, 2023
@lustefaniak
lustefaniak deleted the zdenko/snowflake-parse-trim-with-second-param branch October 27, 2023 16:59
lustefaniak added a commit that referenced this pull request Feb 17, 2026
@lustefaniak
duckdb: support #N positional column references
a7abdbf 
Parse DuckDB's #1, #2 positional column references as Value::Placeholder.
DuckDB tokenizes # as Token::Sharp followed by a number.
lustefaniak added a commit that referenced this pull request Feb 17, 2026
@lustefaniak
duckdb: support #N positional column references
64f3d2e 
Parse DuckDB's #1, #2 positional column references as Value::Placeholder.
DuckDB tokenizes # as Token::Sharp followed by a number.
lustefaniak added a commit that referenced this pull request Feb 17, 2026
@lustefaniak
duckdb: support #N positional column references
78a1796 
Parse DuckDB's #1, #2 positional column references as Value::Placeholder.
DuckDB tokenizes # as Token::Sharp followed by a number.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

See More

@zdenal zdenal

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@zdenal