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/23 02:15:00 UTC
[jira] [Created] (ARROW-13439) [C++] Unify BasicDecimal128 and
BasicDecimal256 classes
Yibo Cai created ARROW-13439:
--------------------------------
Summary: [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
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)