Skip to content

BigQuery: Add support for CREATE SCHEMA options #1742

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 14, 2025

Conversation

iffyio
Copy link
Contributor

@iffyio iffyio commented Feb 24, 2025

CREATE SCHEMA DEFAULT COLLATE 'foo' myschema OPTIONS(key='value');

BigQuery

@iffyio iffyio changed the title Add support for CREATE SCHEMA options BigQuery: Add support for CREATE SCHEMA options Feb 24, 2025
@iffyio iffyio force-pushed the bq-schema-options branch from 426b151 to 64ef5ab Compare March 13, 2025 07:53
@iffyio iffyio requested a review from alamb March 13, 2025 07:57
@@ -988,6 +988,7 @@ fn parse_create_schema_if_not_exists() {
Statement::CreateSchema {
if_not_exists: true,
schema_name,
..
Copy link
Contributor

Choose a reason for hiding this comment

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

I personally think adding explicit ignores here

Suggested change
..
options: _,
default_collate_spec: _,

Is better as then if a new field is added then the compiler will point out this place as needing to be checked

That being said, I don't think it is critical in this case

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@iffyio iffyio force-pushed the bq-schema-options branch from 64ef5ab to 3cd2a47 Compare March 14, 2025 06:46
@iffyio iffyio merged commit f81aed6 into apache:main Mar 14, 2025
9 checks passed
@iffyio iffyio deleted the bq-schema-options branch March 14, 2025 07:00
@alamb
Copy link
Contributor

alamb commented Mar 15, 2025

🚀

ayman-sigma pushed a commit to sigmacomputing/sqlparser-rs that referenced this pull request Apr 10, 2025
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.

2 participants