pepijnve opened a new pull request, #16811: URL: https://github.com/apache/datafusion/pull/16811
## Which issue does this PR close? - Closes #16810. ## Rationale for this change When parsing SQL strings into expressions it's preferable to get parse errors when unprocessed input is present rather than silently ignoring this. ## What changes are included in this PR? - Add `DFParser::parse_into_expr` which expects to consume the entire input SQL string and reports an error if it did not - Adjust `SessionState::sql_to_expr_with_alias` to use this new function. ## Are these changes tested? Yes, extra test cases have been added to `DFParser` ## Are there any user-facing changes? This change makes the preconditions for `SessionState::sql_to_expr` and `SessionState::sql_to_expr_with_alias` stricter which may cause errors to be reported in code that currently completes successfully. -- 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. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
