Skip to content

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
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