-
Notifications
You must be signed in to change notification settings - Fork 4k
Closed
Description
Currently the output type of the Add function is identical with the argument types which makes it unsafe to add numeric limit values, so instead of using (int8, int8) -> int8 signature we should use ((int8, int8) -> int16.
The current kernels do wrap around for performance reasons, so implement different kernels with overflow checking and add arithmetics options to let the caller choose which variant to run
Reporter: Krisztian Szucs / @kszucs
Assignee: Krisztian Szucs / @kszucs
Related issues:
- [R] Add bindings for arithmetic kernels (is depended upon by)
PRs and other links:
Note: This issue was originally created as ARROW-9022. Please see the migration documentation for further details.