-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[POC] Try to plan ast::Expr::CompoundFieldAccess syntax #13734
base: main
Are you sure you want to change the base?
Conversation
Occurred stack overflow when planning tpcds q64 in some os. 🤔 It passed in my local environments.
|
Is the code compiled with opt-level 1? AFAIU, otherwise all enums have insane (stack) size. |
I guess no 🤔 . I didn't see any release-profile argument in the CI workflow. The |
Is the default stack size same for Linux & Windows runners? |
Which issue does this PR close?
Related to apache/datafusion-sqlparser-rs#1551
Rationale for this change
This PR is based on #13546
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?