+1 (non-binding) Checked spec change and C++ impl.
On Wed, Jul 24, 2024 at 6:52 PM Joel Lubinitsky <joell...@gmail.com> wrote: > +1 (non-binding) > > Go implementation LGTM > > On Wed, Jul 24, 2024 at 5:12 AM Raúl Cumplido <rau...@apache.org> wrote: > > > +1 (binding) > > > > Format change looks good to me. I haven't reviewed the individual > > implementations. > > > > Thanks David for leading this. > > > > El mié, 24 jul 2024 a las 10:51, Joris Van den Bossche > > (<jorisvandenboss...@gmail.com>) escribió: > > > > > > +1 (binding) > > > > > > On Wed, 24 Jul 2024 at 07:34, David Li <lidav...@apache.org> wrote: > > > > > > > > Hello, > > > > > > > > I'd like to propose the 'Opaque' canonical extension type. Prior > > discussion can be found at [1] and the proposal and implementations for > > C++, Go, Java, and Python can be found at [2]. The proposal is > additionally > > reproduced below. > > > > > > > > The vote will be open for at least 72 hours. > > > > > > > > [ ] +1 Accept this proposal > > > > [ ] +0 > > > > [ ] -1 Do not accept this proposal because... > > > > > > > > [1]: > https://lists.apache.org/thread/8d5ldl5cb7mms21rd15lhpfrv4j9no4n > > > > [2]: https://github.com/apache/arrow/pull/41823 > > > > > > > > --- > > > > > > > > Opaque represents a type that an Arrow-based system received from an > > external > > > > (often non-Arrow) system, but that it cannot interpret. In this > case, > > it can > > > > pass on Opaque to its clients to at least show that a field exists > and > > > > preserve metadata about the type from the other system. > > > > > > > > Extension parameters: > > > > > > > > * Extension name: ``arrow.opaque``. > > > > > > > > * The storage type of this extension is any type. If there is no > > underlying > > > > data, the storage type should be Null. > > > > > > > > * Extension type parameters: > > > > > > > > * **type_name** = the name of the unknown type in the external > > system. > > > > * **vendor_name** = the name of the external system. > > > > > > > > * Description of the serialization: > > > > > > > > A valid JSON object containing the parameters as fields. In the > > future, > > > > additional fields may be added, but all fields current and future > > are never > > > > required to interpret the array. > > > > > > > > Developers **should not** attempt to enable public semantic > > interoperability > > > > of Opaque by canonicalizing specific values of these parameters. > > >
