Skip to content

Support bigquery CAST AS x [STRING|DATE] FORMAT syntax - #978

Merged
alamb merged 1 commit into
apache:mainfrom
getsynq:cast-format
Oct 20, 2023
Merged

Support bigquery CAST AS x [STRING|DATE] FORMAT syntax#978
alamb merged 1 commit into
apache:mainfrom
getsynq:cast-format

Conversation

@lustefaniak

@lustefaniaklustefaniak commented Sep 24, 2023

Copy link
Copy Markdown
Contributor

@lustefaniak
lustefaniakforce-pushed the cast-format branch 2 times, most recently from 0d42a8a to c274bcdCompareSeptember 25, 2023 15:50

@alambalamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @lustefaniak -- sorry I missed this one in the sea of reiviews. I think it is looking good in my mind -- I have some suggestions -- let me know what you think

Comment threadsrc/ast/mod.rs Outdated
Comment threadsrc/ast/mod.rs Outdated
@alamb

alamb commented Oct 5, 2023

Copy link
Copy Markdown
Contributor

I think the CI failures are not related to the changes in this PR -- if you merge up / rebase from main they should be solved. See #995

@lustefaniak
lustefaniakforce-pushed the cast-format branch 2 times, most recently from ebe32db to 00bd9f3CompareOctober 20, 2023 09:01
@alambalamb changed the title bigquery: CAST AS x FORMAT supportSupport bigquery CAST AS x FORMAT syntaxOct 20, 2023
@alambalamb changed the title Support bigquery CAST AS x FORMAT syntaxSupport bigquery CAST AS x [STRING|DATE] FORMAT syntaxOct 20, 2023

@alambalamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great -- thank you @lustefaniak

@alamb
alamb merged commit c68e977 into apache:mainOct 20, 2023
@lustefaniak
lustefaniak deleted the cast-format branch October 20, 2023 19:14
serprex pushed a commit to serprex/sqlparser-rs that referenced this pull request Nov 6, 2023
Sign up for freeto 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.

2 participants

@lustefaniak@alamb