Skip to content

Commit

Permalink
chore(deps): update sqlparser requirement from 0.35 to 0.36.1 (#7051)
Browse files Browse the repository at this point in the history
* chore(deps): update sqlparser requirement from 0.35 to 0.36

Updates the requirements on [sqlparser](https://github.com/sqlparser-rs/sqlparser-rs) to permit the latest version.
- [Changelog](https://github.com/sqlparser-rs/sqlparser-rs/blob/main/CHANGELOG.md)
- [Commits](apache/datafusion-sqlparser-rs@v0.35.0...v0.35.0)

---
updated-dependencies:
- dependency-name: sqlparser
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update for new API

* Update sqlparser to 0.36.1

* Update datafusion-cli Cargo.lock

* Update test output

* fmt

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
  • Loading branch information
alamb and dependabot[bot] authored Jul 24, 2023
1 parent 9445aa0 commit d2d506a
Show file tree
Hide file tree
Showing 4 changed files with 48 additions and 40 deletions.
2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ arrow-buffer = { version = "43.0.0", default-features = false }
arrow-flight = { version = "43.0.0", features = ["flight-sql-experimental"] }
arrow-schema = { version = "43.0.0", default-features = false }
parquet = { version = "43.0.0", features = ["arrow", "async", "object_store"] }
sqlparser = { version = "0.35", features = ["visitor"] }
sqlparser = { version = "0.36.1", features = ["visitor"] }

[profile.release]
codegen-units = 1
Expand Down
70 changes: 35 additions & 35 deletions datafusion-cli/Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 2 additions & 4 deletions datafusion/core/src/execution/context.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2175,13 +2175,11 @@ mod tests {
async fn create_variable_err() -> Result<()> {
let ctx = SessionContext::new();

let err = plan_and_collect(&ctx, "SElECT @= X#=?!~ 5")
.await
.unwrap_err();
let err = plan_and_collect(&ctx, "SElECT @= X3").await.unwrap_err();

assert_eq!(
err.to_string(),
"Error during planning: variable [\"@\"] has no type information"
"Error during planning: variable [\"@=\"] has no type information"
);
Ok(())
}
Expand Down
10 changes: 10 additions & 0 deletions datafusion/sql/src/set_expr.rs
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,16 @@ impl<'a, S: ContextProvider> SqlToRel<'a, S> {
let all = match set_quantifier {
SetQuantifier::All => true,
SetQuantifier::Distinct | SetQuantifier::None => false,
SetQuantifier::ByName => {
return Err(DataFusionError::NotImplemented(
"UNION BY NAME not implemented".to_string(),
));
}
SetQuantifier::AllByName => {
return Err(DataFusionError::NotImplemented(
"UNION ALL BY NAME not implemented".to_string(),
))
}
};

let left_plan = self.set_expr_to_plan(*left, planner_context)?;
Expand Down

0 comments on commit d2d506a

Please sign in to comment.