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

Posted to jira@arrow.apache.org by "Yibo Cai (Jira)" <ji...@apache.org> on 2021/07/27 07:53:00 UTC

[jira] [Closed] (ARROW-13439) [C++] Unify BasicDecimal128 and BasicDecimal256 classes

     [ https://issues.apache.org/jira/browse/ARROW-13439?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Yibo Cai closed ARROW-13439.
----------------------------
    Fix Version/s:     (was: 6.0.0)
       Resolution: Won't Do

> [C++] Unify BasicDecimal128 and BasicDecimal256 classes
> -------------------------------------------------------
>
>                 Key: ARROW-13439
>                 URL: https://issues.apache.org/jira/browse/ARROW-13439
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Yibo Cai
>            Assignee: Yibo Cai
>            Priority: Major
>
> BasicDecimal128 uses two dedicated int64 (higher, lower) to store the value. It's possible to use std::array<uint64, 2> to store the value, just like BasicDecimal256 does, and to unify the two classes.
> As decimal operations are quite expensive, guess this indirection won't cause performance loss.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)