alamb opened a new pull request, #23: URL: https://github.com/apache/datafusion-testing/pull/23
## Rationale Apache DataFusion PR apache/datafusion#25278 upgrades to `sqlparser` 0.63.0, which adds recursion guards to more parse functions (data type and `INTERVAL` parsing). The expression prefix parser speculatively tries a data type at every expression leaf, so every expression now consumes one more level of the parser recursion budget than it did with `sqlparser` 0.62. To avoid rejecting queries that previously parsed, that PR raises the default `datafusion.sql_parser.recursion_limit` from 50 to 51. The SQLite compatibility tests embed the limit in their expected error messages, so they must be updated to match. ## Changes Update 275 `RecursionLimitExceeded` error-message expectations across 15 SQLite test files from `(current limit: 50)` to `(current limit: 51)`. The SQL queries are unchanged, and every affected query still fails with the new limit (they all need a limit of 52 or more to parse), so only the limit reported in the message changes. ## Testing Pointed DataFusion's `datafusion-testing` submodule at this commit, with apache/datafusion#25278 plus the default set to 51, and ran: ``` INCLUDE_SQLITE=true cargo test --profile release-nonlto --test sqllogictests ``` All 1,115 files completed successfully. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
