We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The feature request is based on the need that I would like get_field(expr, key) to displayed as expr[key].
get_field(expr, key)
expr[key]
two reason
Expr::GetIndexedField
GetFieldAccess
get_field
datafusion/datafusion/expr/src/expr.rs
Lines 1583 to 1595 in 8190cb9
Given the current function to build the name has only a fixed pattern, I think enable user-defined display is a good idea
Lines 1646 to 1653 in 8190cb9
No response
Adding display_name to ScalarUDFImpl
display_name
The text was updated successfully, but these errors were encountered:
Looks like it doesn't need a lot of changes. Since I worked on another issue related to display_name, I want to take this. :)
Sorry, something went wrong.
I'm working on it and I found after #10325 is merged, it will avoid a conflict. Given it's close to be merged, I think we can wait a moment.
Successfully merging a pull request may close this issue.
Is your feature request related to a problem or challenge?
The feature request is based on the need that I would like
get_field(expr, key)
to displayed asexpr[key]
.two reason
Expr::GetIndexedField
andGetFieldAccess
and always use functionget_field
for indexing #10374expr[key]
looks nicer!datafusion/datafusion/expr/src/expr.rs
Lines 1583 to 1595 in 8190cb9
Given the current function to build the name has only a fixed pattern, I think enable user-defined display is a good idea
datafusion/datafusion/expr/src/expr.rs
Lines 1646 to 1653 in 8190cb9
Describe the solution you'd like
No response
Adding
display_name
to ScalarUDFImplDescribe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: