+1 (non-bonding). Thanks.
*Regards,* *Rossi SUN* Joris Van den Bossche <jorisvandenboss...@gmail.com>于2024年9月6日 周五14:46写道： > +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 > > > >
