You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@arrow.apache.org by Micah Kornfield <em...@gmail.com> on 2019/08/16 05:44:01 UTC

[VOTE] Proposed addition to Arrow Flight Protocol

Hello,
Ryan Murray has proposed adding a GetFlightSchema RPC [1] to the Arrow
Flight Protocol [2].  The purpose of this RPC is to allow decoupling schema
and endpoint retrieval as provided by the GetFlightInfo RPC.  The new
definition provided is:

message SchemaResult {
  // Serialized Flatbuffer Schema message.
  bytes schema = 1;
}
rpc GetSchema(FlightDescriptor) returns (SchemaResult) {}

Ryan has also provided a PR demonstrating implementation of the new RPC [3]
in Java, C++ and Python which can be reviewed and merged after this
addition is approved.

Please vote whether to accept the addition. The vote will be open for at
least 72 hours.

[ ] +1 Accept this addition to the Flight protocol
[ ] +0
[ ] -1 Do not accept the changes because...


Thanks,
Micah

[1]
https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-Ef6OJy7H90MqViA/edit
[2] https://github.com/apache/arrow/blob/master/format/Flight.proto
[3] https://github.com/apache/arrow/pull/4980

Re: [VOTE] Proposed addition to Arrow Flight Protocol

Posted by Ryan Murray <ry...@dremio.com>.
Thanks Micah/All!

On Tue, Aug 20, 2019 at 6:06 AM Micah Kornfield <em...@gmail.com>
wrote:

> The motion carries with 4 binding +1 votes,  2 non-binding +1 votes  and no
> other votes.
>
> I think the next step is to review and merge the patch pending patch [1].
>
> Thanks,
> Micah
>
> [1] https://github.com/apache/arrow/pull/4980
>
>
>
> On Mon, Aug 19, 2019 at 2:52 AM Antoine Pitrou <an...@python.org> wrote:
>
> >
> > +1 (binding)
> >
> > Regards
> >
> > Antoine.
> >
> >
> > Le 16/08/2019 à 07:44, Micah Kornfield a écrit :
> > > Hello,
> > > Ryan Murray has proposed adding a GetFlightSchema RPC [1] to the Arrow
> > > Flight Protocol [2].  The purpose of this RPC is to allow decoupling
> > schema
> > > and endpoint retrieval as provided by the GetFlightInfo RPC.  The new
> > > definition provided is:
> > >
> > > message SchemaResult {
> > >   // Serialized Flatbuffer Schema message.
> > >   bytes schema = 1;
> > > }
> > > rpc GetSchema(FlightDescriptor) returns (SchemaResult) {}
> > >
> > > Ryan has also provided a PR demonstrating implementation of the new RPC
> > [3]
> > > in Java, C++ and Python which can be reviewed and merged after this
> > > addition is approved.
> > >
> > > Please vote whether to accept the addition. The vote will be open for
> at
> > > least 72 hours.
> > >
> > > [ ] +1 Accept this addition to the Flight protocol
> > > [ ] +0
> > > [ ] -1 Do not accept the changes because...
> > >
> > >
> > > Thanks,
> > > Micah
> > >
> > > [1]
> > >
> >
> https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-Ef6OJy7H90MqViA/edit
> > > [2] https://github.com/apache/arrow/blob/master/format/Flight.proto
> > > [3] https://github.com/apache/arrow/pull/4980
> > >
> >
>


-- 

Ryan Murray  | Principal Consulting Engineer

+447540852009 | rymurr@dremio.com

<https://www.dremio.com/>
Check out our GitHub <https://www.github.com/dremio>, join our community
site <https://community.dremio.com/> & Download Dremio
<https://www.dremio.com/download>

Re: [VOTE] Proposed addition to Arrow Flight Protocol

Posted by Micah Kornfield <em...@gmail.com>.
The motion carries with 4 binding +1 votes,  2 non-binding +1 votes  and no
other votes.

I think the next step is to review and merge the patch pending patch [1].

Thanks,
Micah

[1] https://github.com/apache/arrow/pull/4980



On Mon, Aug 19, 2019 at 2:52 AM Antoine Pitrou <an...@python.org> wrote:

>
> +1 (binding)
>
> Regards
>
> Antoine.
>
>
> Le 16/08/2019 à 07:44, Micah Kornfield a écrit :
> > Hello,
> > Ryan Murray has proposed adding a GetFlightSchema RPC [1] to the Arrow
> > Flight Protocol [2].  The purpose of this RPC is to allow decoupling
> schema
> > and endpoint retrieval as provided by the GetFlightInfo RPC.  The new
> > definition provided is:
> >
> > message SchemaResult {
> >   // Serialized Flatbuffer Schema message.
> >   bytes schema = 1;
> > }
> > rpc GetSchema(FlightDescriptor) returns (SchemaResult) {}
> >
> > Ryan has also provided a PR demonstrating implementation of the new RPC
> [3]
> > in Java, C++ and Python which can be reviewed and merged after this
> > addition is approved.
> >
> > Please vote whether to accept the addition. The vote will be open for at
> > least 72 hours.
> >
> > [ ] +1 Accept this addition to the Flight protocol
> > [ ] +0
> > [ ] -1 Do not accept the changes because...
> >
> >
> > Thanks,
> > Micah
> >
> > [1]
> >
> https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-Ef6OJy7H90MqViA/edit
> > [2] https://github.com/apache/arrow/blob/master/format/Flight.proto
> > [3] https://github.com/apache/arrow/pull/4980
> >
>

Re: [VOTE] Proposed addition to Arrow Flight Protocol

Posted by Antoine Pitrou <an...@python.org>.
+1 (binding)

Regards

Antoine.


Le 16/08/2019 à 07:44, Micah Kornfield a écrit :
> Hello,
> Ryan Murray has proposed adding a GetFlightSchema RPC [1] to the Arrow
> Flight Protocol [2].  The purpose of this RPC is to allow decoupling schema
> and endpoint retrieval as provided by the GetFlightInfo RPC.  The new
> definition provided is:
> 
> message SchemaResult {
>   // Serialized Flatbuffer Schema message.
>   bytes schema = 1;
> }
> rpc GetSchema(FlightDescriptor) returns (SchemaResult) {}
> 
> Ryan has also provided a PR demonstrating implementation of the new RPC [3]
> in Java, C++ and Python which can be reviewed and merged after this
> addition is approved.
> 
> Please vote whether to accept the addition. The vote will be open for at
> least 72 hours.
> 
> [ ] +1 Accept this addition to the Flight protocol
> [ ] +0
> [ ] -1 Do not accept the changes because...
> 
> 
> Thanks,
> Micah
> 
> [1]
> https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-Ef6OJy7H90MqViA/edit
> [2] https://github.com/apache/arrow/blob/master/format/Flight.proto
> [3] https://github.com/apache/arrow/pull/4980
> 

Re: [VOTE] Proposed addition to Arrow Flight Protocol

Posted by Sutou Kouhei <ko...@clear-code.com>.
+1 (binding)

In <CA...@mail.gmail.com>
  "Re: [VOTE] Proposed addition to Arrow Flight Protocol" on Fri, 16 Aug 2019 11:51:14 -0500,
  Wes McKinney <we...@gmail.com> wrote:

> +1 (binding)
> 
> On Fri, Aug 16, 2019 at 11:34 AM Bryan Cutler <cu...@gmail.com> wrote:
>>
>> +1 (non-binding)
>>
>> On Fri, Aug 16, 2019, 8:36 AM Micah Kornfield <em...@gmail.com> wrote:
>>
>> > My vote +1 (binding)
>> >
>> > On Friday, August 16, 2019, David Li <li...@gmail.com> wrote:
>> >
>> > > +1 (non-binding)
>> > >
>> > > Thanks Ryan for working on this!
>> > >
>> > > Best,
>> > > David
>> > >
>> > > On 8/16/19, Micah Kornfield <em...@gmail.com> wrote:
>> > > > Hello,
>> > > > Ryan Murray has proposed adding a GetFlightSchema RPC [1] to the Arrow
>> > > > Flight Protocol [2].  The purpose of this RPC is to allow decoupling
>> > > schema
>> > > > and endpoint retrieval as provided by the GetFlightInfo RPC.  The new
>> > > > definition provided is:
>> > > >
>> > > > message SchemaResult {
>> > > >   // Serialized Flatbuffer Schema message.
>> > > >   bytes schema = 1;
>> > > > }
>> > > > rpc GetSchema(FlightDescriptor) returns (SchemaResult) {}
>> > > >
>> > > > Ryan has also provided a PR demonstrating implementation of the new RPC
>> > > [3]
>> > > > in Java, C++ and Python which can be reviewed and merged after this
>> > > > addition is approved.
>> > > >
>> > > > Please vote whether to accept the addition. The vote will be open for
>> > at
>> > > > least 72 hours.
>> > > >
>> > > > [ ] +1 Accept this addition to the Flight protocol
>> > > > [ ] +0
>> > > > [ ] -1 Do not accept the changes because...
>> > > >
>> > > >
>> > > > Thanks,
>> > > > Micah
>> > > >
>> > > > [1]
>> > > > https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-
>> > > Ef6OJy7H90MqViA/edit
>> > > > [2] https://github.com/apache/arrow/blob/master/format/Flight.proto
>> > > > [3] https://github.com/apache/arrow/pull/4980
>> > > >
>> > >
>> >

Re: [VOTE] Proposed addition to Arrow Flight Protocol

Posted by Wes McKinney <we...@gmail.com>.
+1 (binding)

On Fri, Aug 16, 2019 at 11:34 AM Bryan Cutler <cu...@gmail.com> wrote:
>
> +1 (non-binding)
>
> On Fri, Aug 16, 2019, 8:36 AM Micah Kornfield <em...@gmail.com> wrote:
>
> > My vote +1 (binding)
> >
> > On Friday, August 16, 2019, David Li <li...@gmail.com> wrote:
> >
> > > +1 (non-binding)
> > >
> > > Thanks Ryan for working on this!
> > >
> > > Best,
> > > David
> > >
> > > On 8/16/19, Micah Kornfield <em...@gmail.com> wrote:
> > > > Hello,
> > > > Ryan Murray has proposed adding a GetFlightSchema RPC [1] to the Arrow
> > > > Flight Protocol [2].  The purpose of this RPC is to allow decoupling
> > > schema
> > > > and endpoint retrieval as provided by the GetFlightInfo RPC.  The new
> > > > definition provided is:
> > > >
> > > > message SchemaResult {
> > > >   // Serialized Flatbuffer Schema message.
> > > >   bytes schema = 1;
> > > > }
> > > > rpc GetSchema(FlightDescriptor) returns (SchemaResult) {}
> > > >
> > > > Ryan has also provided a PR demonstrating implementation of the new RPC
> > > [3]
> > > > in Java, C++ and Python which can be reviewed and merged after this
> > > > addition is approved.
> > > >
> > > > Please vote whether to accept the addition. The vote will be open for
> > at
> > > > least 72 hours.
> > > >
> > > > [ ] +1 Accept this addition to the Flight protocol
> > > > [ ] +0
> > > > [ ] -1 Do not accept the changes because...
> > > >
> > > >
> > > > Thanks,
> > > > Micah
> > > >
> > > > [1]
> > > > https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-
> > > Ef6OJy7H90MqViA/edit
> > > > [2] https://github.com/apache/arrow/blob/master/format/Flight.proto
> > > > [3] https://github.com/apache/arrow/pull/4980
> > > >
> > >
> >

Re: [VOTE] Proposed addition to Arrow Flight Protocol

Posted by Bryan Cutler <cu...@gmail.com>.
+1 (non-binding)

On Fri, Aug 16, 2019, 8:36 AM Micah Kornfield <em...@gmail.com> wrote:

> My vote +1 (binding)
>
> On Friday, August 16, 2019, David Li <li...@gmail.com> wrote:
>
> > +1 (non-binding)
> >
> > Thanks Ryan for working on this!
> >
> > Best,
> > David
> >
> > On 8/16/19, Micah Kornfield <em...@gmail.com> wrote:
> > > Hello,
> > > Ryan Murray has proposed adding a GetFlightSchema RPC [1] to the Arrow
> > > Flight Protocol [2].  The purpose of this RPC is to allow decoupling
> > schema
> > > and endpoint retrieval as provided by the GetFlightInfo RPC.  The new
> > > definition provided is:
> > >
> > > message SchemaResult {
> > >   // Serialized Flatbuffer Schema message.
> > >   bytes schema = 1;
> > > }
> > > rpc GetSchema(FlightDescriptor) returns (SchemaResult) {}
> > >
> > > Ryan has also provided a PR demonstrating implementation of the new RPC
> > [3]
> > > in Java, C++ and Python which can be reviewed and merged after this
> > > addition is approved.
> > >
> > > Please vote whether to accept the addition. The vote will be open for
> at
> > > least 72 hours.
> > >
> > > [ ] +1 Accept this addition to the Flight protocol
> > > [ ] +0
> > > [ ] -1 Do not accept the changes because...
> > >
> > >
> > > Thanks,
> > > Micah
> > >
> > > [1]
> > > https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-
> > Ef6OJy7H90MqViA/edit
> > > [2] https://github.com/apache/arrow/blob/master/format/Flight.proto
> > > [3] https://github.com/apache/arrow/pull/4980
> > >
> >
>

Re: [VOTE] Proposed addition to Arrow Flight Protocol

Posted by Micah Kornfield <em...@gmail.com>.
My vote +1 (binding)

On Friday, August 16, 2019, David Li <li...@gmail.com> wrote:

> +1 (non-binding)
>
> Thanks Ryan for working on this!
>
> Best,
> David
>
> On 8/16/19, Micah Kornfield <em...@gmail.com> wrote:
> > Hello,
> > Ryan Murray has proposed adding a GetFlightSchema RPC [1] to the Arrow
> > Flight Protocol [2].  The purpose of this RPC is to allow decoupling
> schema
> > and endpoint retrieval as provided by the GetFlightInfo RPC.  The new
> > definition provided is:
> >
> > message SchemaResult {
> >   // Serialized Flatbuffer Schema message.
> >   bytes schema = 1;
> > }
> > rpc GetSchema(FlightDescriptor) returns (SchemaResult) {}
> >
> > Ryan has also provided a PR demonstrating implementation of the new RPC
> [3]
> > in Java, C++ and Python which can be reviewed and merged after this
> > addition is approved.
> >
> > Please vote whether to accept the addition. The vote will be open for at
> > least 72 hours.
> >
> > [ ] +1 Accept this addition to the Flight protocol
> > [ ] +0
> > [ ] -1 Do not accept the changes because...
> >
> >
> > Thanks,
> > Micah
> >
> > [1]
> > https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-
> Ef6OJy7H90MqViA/edit
> > [2] https://github.com/apache/arrow/blob/master/format/Flight.proto
> > [3] https://github.com/apache/arrow/pull/4980
> >
>

Re: [VOTE] Proposed addition to Arrow Flight Protocol

Posted by David Li <li...@gmail.com>.
+1 (non-binding)

Thanks Ryan for working on this!

Best,
David

On 8/16/19, Micah Kornfield <em...@gmail.com> wrote:
> Hello,
> Ryan Murray has proposed adding a GetFlightSchema RPC [1] to the Arrow
> Flight Protocol [2].  The purpose of this RPC is to allow decoupling schema
> and endpoint retrieval as provided by the GetFlightInfo RPC.  The new
> definition provided is:
>
> message SchemaResult {
>   // Serialized Flatbuffer Schema message.
>   bytes schema = 1;
> }
> rpc GetSchema(FlightDescriptor) returns (SchemaResult) {}
>
> Ryan has also provided a PR demonstrating implementation of the new RPC [3]
> in Java, C++ and Python which can be reviewed and merged after this
> addition is approved.
>
> Please vote whether to accept the addition. The vote will be open for at
> least 72 hours.
>
> [ ] +1 Accept this addition to the Flight protocol
> [ ] +0
> [ ] -1 Do not accept the changes because...
>
>
> Thanks,
> Micah
>
> [1]
> https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-Ef6OJy7H90MqViA/edit
> [2] https://github.com/apache/arrow/blob/master/format/Flight.proto
> [3] https://github.com/apache/arrow/pull/4980
>