-
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
Use lowercase name
for aggregate function
#10695
Labels
enhancement
New feature or request
Comments
I think we need to rename other builtin aggregate function to lowercase too. |
jayzhan211
changed the title
Remove redundant upper case aliases for
Use lowercase May 30, 2024
FIRST_VALUE
and LAST_VALUE
name
for aggregate function
This was referenced Jun 3, 2024
Merged
This was referenced Jun 14, 2024
This might be solved too? @alamb |
Yeah, solved |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem or challenge?
first_value
andlast_value
window functions currently report their names asFIRST_VALUE
but also have a lower case alias:datafusion/datafusion/functions-aggregate/src/first_last.rs
Line 75 in 98cd19e
part of #8708
Describe the solution you'd like
I think we would like to avoid the aliases and simply have a single name, similarly how the other functions work
TODO
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: