returnString opened a new pull request #214:
URL: https://github.com/apache/arrow-datafusion/pull/214
Some background first! @Dandandan did the initial work here and identified
the VarProvider stuff as a potential followup issue in apache/arrow#9541. This
PR just implements the original dialect change with the proposed extra change
of removing scalar vars - thanks Daniel! 😄
# Which issue does this PR close?
Closes #183.
# Rationale for this change
This formalises our decision to use the Postgres SQL dialect.
# What changes are included in this PR?
- The DataFusion SQL parser now uses the Postgres dialect from `sqlparser-rs`
- The notion of ScalarVariable exprs and the related VarProvider concept
have been removed entirely, as these aren't supported in the Postgres dialect
and the use case can be handled with nullary functions [as is done in
Postgres](https://www.postgresql.org/docs/current/functions-info.html).
# Are there any user-facing changes?
DataFusion (unsure if this worked before in Ballista) no longer support user
or system variables exposed with the non-standard `@` prefix.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
