[Data] Compute Expressions-Expr Rounding#59295
Conversation
Signed-off-by: yaommen <myanstu@163.com>
94c1046 to
7081bb0
Compare
|
Hi, @goutamvenkat-anyscale ,Could you please review this PR when you have time?many thanks |
|
hey @myandpr, thanks a bunch for the contribution! if you're interested in chatting more with the contributors, feel free to join our community sync -- https://docs.google.com/forms/d/e/1FAIpQLSeYWjNExnr6gbhO5rpM0i6wm4TBTdsm3y5S0LR8Syzk_2gelQ/viewform |
@richardliaw Submitted the form! Looking forward to the next sync meeting.Looking forward to contributing more and learning from everyone. |
|
@bveeramani PTAL |
goutamvenkat-anyscale
left a comment
There was a problem hiding this comment.
Thanks for your contribution! Just a few comments
bveeramani
left a comment
There was a problem hiding this comment.
LGTM other than Goutam's suggestions
Signed-off-by: yaommen <myanstu@163.com>
goutamvenkat-anyscale
left a comment
There was a problem hiding this comment.
lgtm! Thanks
Description
Completing the Expr (direct) Rounding operations (ceil, floor, round, trunc)
for example:
Related issues
Related to #58674
Additional information