Re: [DISCUSS] Add GetFlightSchema to Flight RPC

Thanks both :-) On Fri, Aug 16, 2019 at 6:22 AM Micah Kornfield wrote: > I'll start one shortly. > > On Thu, Aug 15, 2019 at 4:31 PM Wes McKinney wrote: > > > Yes, I think having a vote as a procedural matter would be a good thing. > > > > I have run dozens of public and private votes in my rol

Re: [DISCUSS] Add GetFlightSchema to Flight RPC

I'll start one shortly. On Thu, Aug 15, 2019 at 4:31 PM Wes McKinney wrote: > Yes, I think having a vote as a procedural matter would be a good thing. > > I have run dozens of public and private votes in my role as a PMC > member. I would appreciate if another PMC would assist with this vote. >

Re: [DISCUSS] Add GetFlightSchema to Flight RPC

Yes, I think having a vote as a procedural matter would be a good thing. I have run dozens of public and private votes in my role as a PMC member. I would appreciate if another PMC would assist with this vote. Thanks On Wed, Aug 14, 2019 at 5:37 PM Ryan Murray wrote: > > Hi All, > > Does this r

Re: [DISCUSS] Add GetFlightSchema to Flight RPC

Hi All, Does this require a vote? If yes what is the process for initiating one & if no I hope this is enough time for feedback and I would like to remove the draft designation from the PR Best, Ryan On Wed, Aug 7, 2019 at 9:31 AM Ryan Murray wrote: > As per everyone's feedback I have renamed

Re: [DISCUSS] Add GetFlightSchema to Flight RPC

As per everyone's feedback I have renamed GetFlightSchema -> GetSchema and have removed the descriptor on the rpc result message. The doc has been updated as has the draft PR On Thu, Aug 1, 2019 at 6:32 PM Bryan Cutler wrote: > Sounds good to me, I would just echo what others have said. > > On T

Re: [DISCUSS] Add GetFlightSchema to Flight RPC

Sounds good to me, I would just echo what others have said. On Thu, Aug 1, 2019 at 8:17 AM Ryan Murray wrote: > Thanks Wes, > > The descriptor is only there to maintain a bit of symmetry with > GetFlightInfo. Happy to remove it, I don't think its necessary and already > a few people agree. Simil

Re: [DISCUSS] Add GetFlightSchema to Flight RPC

Thanks Wes, The descriptor is only there to maintain a bit of symmetry with GetFlightInfo. Happy to remove it, I don't think its necessary and already a few people agree. Similar with the method name, I am neutral to the naming and can call it whatever the community is happy with. Best, Ryan On

Re: [DISCUSS] Add GetFlightSchema to Flight RPC

I'm generally supporting of adding the new RPC endpoint. To make a couple points from the document * I'm not sure what the purpose of returning the FlightDescriptor is, but I haven't thought too hard about it * The Schema consists of a single IPC message -- dictionaries will appear in the actual

Re: [DISCUSS] Add GetFlightSchema to Flight RPC

Hi Ryan, Thanks for writing this up! I made a couple of minor comments in the doc/implementation, but overall I'm in favor of having this RPC method. Best, David On 8/1/19, Ryan Murray wrote: > Hi All, > > Please see the attached document for a proposed addition to the Flight > RPC[1]. This is

[DISCUSS] Add GetFlightSchema to Flight RPC

Hi All, Please see the attached document for a proposed addition to the Flight RPC[1]. This is the result of a previous mailing list discussion[2]. I have created the Pull Request[3] to make the proposal a little more concrete. Please let me know if you have any