feat: support 2-argument ceil(value, scale) for Spark compatibility#21562
feat: support 2-argument ceil(value, scale) for Spark compatibility#21562SAY-5 wants to merge 2 commits intoapache:mainfrom
Conversation
Extends the ceil function to accept an optional scale parameter, matching Spark SQL behavior: ceil(value, scale) rounds up to the specified number of decimal places. Implementation: ceil(value, scale) = ceil(value * 10^scale) / 10^scale Supports Float32 and Float64 with Int64 scale, both scalar and array. Closes apache#21560
|
The cargo fmt and docs check failures need the full workspace to resolve locally. I can't run cargo fmt with a sparse checkout. If a maintainer could run |
I don't know what that means but you can run the fmt using |
|
Thanks for the pointer to the scripts. I checked out the full workspace and ran cargo fmt -- no changes needed, the code was already formatted. The docs script needs a full cargo build to generate function tables, which I can't do from a sparse checkout. Could a maintainer run |
Extends
ceilto accept an optional scale parameter for Spark compatibility:ceil(3.145, 2)returns3.15.Implementation is
ceil(value * 10^scale) / 10^scale. Handles Float32/Float64 with Int64 scale, both scalar and array inputs.Closes #21560