-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Update sqlparser requirement from 0.26 to 0.27 #4226
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
Conversation
Updates the requirements on [sqlparser](https://github.com/sqlparser-rs/sqlparser-rs) to permit the latest version. - [Release notes](https://github.com/sqlparser-rs/sqlparser-rs/releases) - [Changelog](https://github.com/sqlparser-rs/sqlparser-rs/blob/main/CHANGELOG.md) - [Commits](apache/datafusion-sqlparser-rs@v0.26.0...v0.27.0) --- updated-dependencies: - dependency-name: sqlparser dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
…ster/sqlparser-0.27
|
This PR is not quite ready -- I need to fix up changes related to apache/datafusion-sqlparser-rs#662 -- but ran out of time todya |
|
|
||
| let results = plan_and_collect( | ||
| &ctx, | ||
| "SELECT array_agg(c13 ORDER BY c1) FROM aggregate_test_100", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is newly added syntax support in apache/datafusion-sqlparser-rs#662
| within_group, | ||
| } = array_agg; | ||
|
|
||
| if let Some(order_by) = order_by { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| all, | ||
| set_quantifier, | ||
| } => { | ||
| let all = match set_quantifier { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sqlparser now distinguishes between ALL , DISTINCT and not specified (previously, not specified left all=false)
|
Benchmark runs are scheduled for baseline = 929a175 and contender = a0581dc. a0581dc is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Which issue does this PR close?
Closes #4201
Rationale for this change
Keep up with dependencies
What changes are included in this PR?
Take @dependabot changes in #4201 and fix up the code for the changes
Update sqlparser requirement from 0.26 to 0.27
Changelog
Sourced from sqlparser's changelog.
... (truncated)
Commits
0d91662(cargo-release) version 0.27.06a84710Update CHANGELOG for 0.27.0 release (#706)814367aImplement ON CONFLICT and RETURNING (#666)ae1c690Fix broken DataFusion link (#703)65c5a37feat: add precision for TIME, DATETIME, and TIMESTAMP data types (#701)cdf4447feat: add FULLTEXT option on create table for MySQL and Generic dialects (#702)87b4a16Parse ARRAY_AGG for Bigquery and Snowflake (#662)0428ac7Add MySql, BigQuery to all dialects (#697)f7817bcSupportDISTINCTfor SetOperator (#689)0f7e144Support the ARRAY type of Snowflake (#699)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.