-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[datafusion-spark] Implement ceil
function
#15916
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
Comments
@shehabgamin and @andygrove -- here is a ticket for another spark function. I am hoping we can do one or two of these functions to set a pattern, and then we will be able to basically copy the pattern for the rest of the library Questions for you:
|
take |
|
Sounds good -- thanks @shehabgamin Looks to me like @irenjj has offered to take a look at this one, so let's see what they come up with Note that @andygrove created a PR recently for |
Is your feature request related to a problem or challenge?
datafusion-spark
Spark Compatible Functions #15914Given the importantance of spark functions in general, we are consolidating / adding spark compatible functions in the
datafusion-spark
crate in DataFusion: https://github.com/apache/datafusion/tree/main/datafusion/sparkThis ticket is part of a series of ticket to fill out the set of spark compatible functions in DataFusion
Describe the solution you'd like
Implement the function listed above in the
datafusion-spark
crateDescribe alternatives you've considered
Examples to follow:
expm1
implementation : https://github.com/apache/datafusion/blob/main/datafusion/spark/src/function/math/expm1.rsYou can also find the corresponding datafusion implementations in the corresponding file here: https://github.com/apache/datafusion/tree/main/datafusion/functions/src (it would be ok to call into these implementations if they are the same as the spark implementation)
Additional context
No response
The text was updated successfully, but these errors were encountered: