-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Move sql_compound_identifier_to_expr
to ExprPlanner
#11473
Comments
seems like a duplicate of #11244 |
take |
sql_compound_identifier_to_expr
to ExprPlanner
QQ: @alamb In |
Makes sense to me 👍 |
After #11487 I wonder if there is anythng left for this item? |
I think we are done with this item |
Is your feature request related to a problem or challenge?
Part of #11207
As discussed in #11207 The idea is to move hardcoded the
sql_compound_identifier_to_expr
operator to function in the parser step, which is currently handled in the optimizer step. We need to implement user-defined parser mechanisms to enable more flexible SQL planning.Describe the solution you'd like
No response
Describe alternatives you've considered
I think you can just follow the model of #11398 and add a function to
ExprPlanner
and then update the sql planner to call it instead ofget_functon_meta
Additional context
No response
The text was updated successfully, but these errors were encountered: