Skip to content

chore: Update sqlparser to 0.63.0 - #25278

Draft
alamb wants to merge 6 commits into
apache:mainfrom
alamb:update-sqlparser-0.63
Draft

alamb wants to merge 6 commits into
apache:mainfrom
alamb:update-sqlparser-0.63

Conversation

@alamb

@alamb alamb commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Rationale for this change

sqlparser 0.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?

  1. Update sqlparser to 0.63.0
  2. Update DataFusion for upstream AST changes

Are these changes tested?

Yes, by existing CI (including sqllogictests). #24926 ran the same code changes against the pre-release sqlparser.

Are there any user-facing changes?

Updated sqlparser version. Error messages for a few unsupported syntax cases have changed slightly (see updated test in datafusion/sql/tests/cases/params.rs).

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
@github-actions github-actions Bot added the sql SQL Planner label Sep 13, 2026
@codecov-commenter

codecov-commenter commented Sep 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.31579% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.90%. Comparing base (15f32dd) to head (e2dfc2a).

Files with missing lines Patch % Lines
datafusion/sql/src/statement.rs 85.00% 6 Missing ⚠️
datafusion/sql/src/expr/mod.rs 60.00% 4 Missing ⚠️
datafusion/sql/src/query.rs 72.72% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #25278      +/-   ##
==========================================
- Coverage   81.90%   81.90%   -0.01%     
==========================================
  Files        1134     1134              
  Lines      425200   425246      +46     
  Branches   425200   425246      +46     
==========================================
+ Hits       348268   348294      +26     
- Misses      56291    56309      +18     
- Partials    20641    20643       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions Bot added the sqllogictest SQL Logic Tests (.slt) label Sep 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sql SQL Planner sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants