Conversation
Pin sqlparser to the head of apache/datafusion-sqlparser-rs (0821cbdd5cbdc27f9c7205466d357deef632c936) and update DataFusion for the AST changes, to validate the upcoming 0.63.0 release. See apache/datafusion-sqlparser-rs#2453
sqlparser now lists the [NOT] JSON alternatives in the expected-token error message after IS
Pin to the head of the release PR (apache/datafusion-sqlparser-rs#2481), which includes the version bump to 0.63.0 plus two AST changes merged since the previous pin: - LIKE / ILIKE / SIMILAR TO escape_char is now Option<Box<Expr>>; DataFusion still only accepts a single-character string literal - New MergeAction::DoNothing variant, rejected with a not implemented error
Picks up sqlparser_derive 0.6.0 and the source-order function clause visitor change (apache/datafusion-sqlparser-rs#2477)
# Conflicts: # datafusion/sql/src/unparser/expr.rs
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #25278 +/- ##
==========================================
- Coverage 81.92% 81.91% -0.01%
==========================================
Files 1135 1135
Lines 427573 427619 +46
Branches 427573 427619 +46
==========================================
+ Hits 350279 350303 +24
- Misses 56367 56385 +18
- Partials 20927 20931 +4 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Resolve conflict in datafusion/sql/src/query.rs: keep main's rewritten ORDER BY ALL handling and clone `OrderByOptions`, which is no longer `Copy` in sqlparser 0.63. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
alamb
marked this pull request as ready for review
September 15, 2026 20:36
Contributor
Author
|
run benchmark sql_planner |
|
🤖 Benchmark running (GKE) | trigger CPU Details (lscpu)Comparing update-sqlparser-0.63 (50f7984) to a0631ed (merge-base) diff Run configurationrun benchmark sql_plannerResults will be posted here when complete File an issue against this benchmark runner |
comphead
reviewed
Sep 15, 2026
| statement error This feature is not implemented: Calling array_agg: LIMIT not supported in function arguments: 1 | ||
| SELECT array_agg(c13 LIMIT 1) FROM aggregate_test_100 | ||
|
|
||
| statement error This feature is not implemented: Calling array_agg: WHERE not supported in function arguments: c2 > 1 |
|
🤖 Benchmark completed (GKE) | trigger Instance: Comparing update-sqlparser-0.63 (50f7984) to a0631ed (merge-base) diff Run configurationrun benchmark sql_plannerCPU Details (lscpu)Details
Resource Usagesql_planner — base (merge-base)
sql_planner — branch
File an issue against this benchmark runner |
comphead
reviewed
Sep 15, 2026
| } | ||
| ast::MergeAction::Delete { .. } => MergeIntoAction::Delete, | ||
| ast::MergeAction::DoNothing { .. } => { | ||
| return not_impl_err!("MERGE DO NOTHING is not supported"); |
Contributor
There was a problem hiding this comment.
this err message is super confusing IMO
comphead
reviewed
Sep 15, 2026
| when not matched then insert *; | ||
|
|
||
| statement error DataFusion error: This feature is not implemented: MERGE DO NOTHING is not supported | ||
| merge into target using source on target.id = source.id |
Contributor
There was a problem hiding this comment.
its useless statement, wondering why it is considering to be a valid sql
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Which issue does this PR close?
0.63.0around 2026-08-31 datafusion-sqlparser-rs#2453Rationale for this change
sqlparser0.63.0 has been released (changelog), so update DataFusion to use it.This PR is based on the testing PR #24926
What changes are included in this PR?
sqlparserto 0.63.0Are these changes tested?
Yes, by existing CI (including sqllogictests).
Are there any user-facing changes?
Updated
sqlparserversion. Error messages for a few unsupported syntax cases have changed slightly (see updated test indatafusion/sql/tests/cases/params.rs).All existing tests continue to pass