Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update recursion limit in DataFusion #14857

Open
Omega359 opened this issue Feb 24, 2025 · 0 comments
Open

Update recursion limit in DataFusion #14857

Omega359 opened this issue Feb 24, 2025 · 0 comments

Comments

@Omega359
Copy link
Contributor

Omega359 commented Feb 24, 2025

As mentioned in #14095 (comment) and discovered in #14091 it seems that we could look into increasing the recursion limit to allow more sql tests to pass without cause stack overflows.

This ticket is to investigate just how much the recursion limit would need to be increased to allow queries such as in #14091 to pass without issues. Once that limit is found we could update DF with that as the default recursion limit.

This change will require changes to DFParser and likely a new config option. The default recursion limit is 50 as seen @ https://github.com/apache/datafusion-sqlparser-rs/blob/aab12add36bfb4dfd60c2ba38682b503cc248199/src/parser/mod.rs#L187

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant