You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@arrow.apache.org by "Antoine Pitrou (Jira)" <ji...@apache.org> on 2022/06/07 07:02:00 UTC

[jira] [Commented] (ARROW-16762) [C++] Implement saturation arithmetic kernels

    [ https://issues.apache.org/jira/browse/ARROW-16762?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17550835#comment-17550835 ] 

Antoine Pitrou commented on ARROW-16762:
----------------------------------------

Saturated arithmetic would probably use dedicated kernels for performance.
I agree it does not make sense to add them unless there's a real world use case.

> [C++] Implement saturation arithmetic kernels
> ---------------------------------------------
>
>                 Key: ARROW-16762
>                 URL: https://issues.apache.org/jira/browse/ARROW-16762
>             Project: Apache Arrow
>          Issue Type: New Feature
>          Components: C++
>            Reporter: Ian Cook
>            Priority: Minor
>
> The Arrow C++ library currently lacks saturation arithmetic kernels. I do not think it is especially important to add them, but I am curious how straightforward it might be. For example could they be implemented by extending the {{_checked}} arithmetic kernels to include an option to emit the min/max of the type when overflow is detected (in each position of the array)?



--
This message was sent by Atlassian Jira
(v8.20.7#820007)