+1 (binding)
On Fri, 6 Sept 2024 at 03:57, Gang Wu <ust...@gmail.com> wrote: > > +1 (non-binding) > > On Fri, Sep 6, 2024 at 3:57 AM Sutou Kouhei <k...@clear-code.com> wrote: > > > +1 (binding) > > > > In <cah4123zv0ph37hrufui1oejzddt3+fwkturk1unpsjmrzuk...@mail.gmail.com> > > "[VOTE] Allow Decimal32 and Decimal64 bitwidths in Arrow Format" on Wed, > > 4 Sep 2024 17:17:49 -0400, > > Matt Topol <zotthewiz...@gmail.com> wrote: > > > > > Based on various discussions among the ecosystem and to continue > > expanding > > > the zero-copy interoperability for Arrow to be used with different > > > libraries and databases (such as libcudf, ClickHouse, etc) I would like > > to > > > propose that we extend the allowable bit-widths for Arrow Decimal types > > to > > > allow 32-bit and 64-bit decimals. > > > > > > The Arrow Spec currently defines the Decimal type as a parameterized > > type, > > > parameterized by the bit-width, and then just specifies that the only > > > allowed bitwidths are 128 and 256. Thus, rather than adding an entirely > > new > > > type we could simply expand what is allowed for the bitwidth field which > > > makes the format side of this a very small change. > > > > > > I've uploaded a PR for adding support for this to C++ [1] and will be > > > uploading a PR for a corresponding Go change within the next day and will > > > respond to this thread with the link. > > > > > > The vote will be open for at least 72 hours. > > > > > > [ ] +1 - Update the Arrow Spec to allow for 32-bit and 64-bit bitwidths > > for > > > Arrow Decimal types > > > [ ] +0 > > > [ ] -1 - Do not update the Arrow Spec to allow for 32-bit and 64-bit > > > bitwidths for Arrow Decimal types because.... > > > > > > Thanks everyone! > > > > > > --Matt > > > > > > [1]: https://github.com/apache/arrow/pull/43957 > >
