+1 (binding) I agree we should be explicit about RFC-8259
On Mon, Apr 29, 2024 at 4:46 PM David Li <lidav...@apache.org> wrote: > +1 (binding) > > assuming we explicitly state RFC-8259 > > On Tue, Apr 30, 2024, at 08:02, Matt Topol wrote: > > +1 (binding) > > > > On Mon, Apr 29, 2024 at 5:36 PM Ian Cook <ianmc...@apache.org> wrote: > > > >> +1 (non-binding) > >> > >> I added a comment in the PR suggesting that we explicitly refer to > RFC-8259 > >> in CanonicalExtensions.rst. > >> > >> On Mon, Apr 29, 2024 at 1:21 PM Micah Kornfield <emkornfi...@gmail.com> > >> wrote: > >> > >> > +1, I added a comment to the PR because I think we should recommend > >> > implementations specifically reject parsing Binary arrays with the > >> > annotation in-case we want to support non-UTF8 encodings in the future > >> > (even thought IIRC these aren't really JSON spec compliant). > >> > > >> > On Fri, Apr 19, 2024 at 1:24 PM Rok Mihevc <rok.mih...@gmail.com> > wrote: > >> > > >> > > Hi all, > >> > > > >> > > Following discussions [1][2] and preliminary implementation work (by > >> > > Pradeep Gollakota) [3] I would like to propose a vote to add > language > >> for > >> > > JSON canonical extension type to CanonicalExtensions.rst as in PR > [4] > >> and > >> > > written below. > >> > > A draft C++ implementation PR can be seen here [3]. > >> > > > >> > > [1] > https://lists.apache.org/thread/p3353oz6lk846pnoq6vk638tjqz2hm1j > >> > > [2] > https://lists.apache.org/thread/7xph3476g9rhl9mtqvn804fqf5z8yoo1 > >> > > [3] https://github.com/apache/arrow/pull/13901 > >> > > [4] https://github.com/apache/arrow/pull/41257 <- proposed change > >> > > > >> > > > >> > > The vote will be open for at least 72 hours. > >> > > > >> > > [ ] +1 Accept this proposal > >> > > [ ] +0 > >> > > [ ] -1 Do not accept this proposal because... > >> > > > >> > > > >> > > JSON > >> > > ==== > >> > > > >> > > * Extension name: `arrow.json`. > >> > > > >> > > * The storage type of this extension is ``StringArray`` or > >> > > or ``LargeStringArray`` or ``StringViewArray``. > >> > > Only UTF-8 encoded JSON is supported. > >> > > > >> > > * Extension type parameters: > >> > > > >> > > This type does not have any parameters. > >> > > > >> > > * Description of the serialization: > >> > > > >> > > Metadata is either an empty string or a JSON string with an empty > >> > object. > >> > > In the future, additional fields may be added, but they are not > >> > required > >> > > to interpret the array. > >> > > > >> > > > >> > > > >> > > Rok > >> > > > >> > > >> >
