edubraqd opened a new pull request, #24914: URL: https://github.com/apache/datafusion/pull/24914
## Which issue does this PR close? - Closes #24913. ## Rationale for this change `exec_and_print` parsed statements with `DFParser::parse_sql_with_dialect`, which always uses the parser's built-in recursion limit, so `SET datafusion.sql_parser.recursion_limit` had no effect in the CLI. ## What changes are included in this PR? Build the parser with `DFParserBuilder`, passing the dialect and the session's `recursion_limit`, the same way `SessionState::sql_to_statement` does. ## Are these changes tested? Yes. `exec_and_print_honours_parser_recursion_limit` runs a 60-level nested expression: it fails with `RecursionLimitExceeded` under the default limit and succeeds once the session limit is raised to 100. ## Are there any user-facing changes? `datafusion.sql_parser.recursion_limit` now applies to statements run through `datafusion-cli`. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
