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 DoGet stream. To motivate why this is -- different endpoints might have different dictionaries corresponding to fields in the schema, to have static/constant dictionaries in a distributed Flight setting is likely to be impractical. I summarize the issue as "dictionaries are data, not metadata". * I would be OK calling this GetSchema instead of GetFlightSchema but either is okay - Wes On Thu, Aug 1, 2019 at 8:08 AM David Li <li.david...@gmail.com> wrote: > > 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 <rym...@dremio.com> wrote: > > 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. > > <https://www.dremio.com/download> > > Please let me know if you have any questions or concerns. > > > > Best, > > Ryan > > > > [1]: > > https://docs.google.com/document/d/1zLdFYikk3owbKpHvJrARLMlmYpi-Ef6OJy7H90MqViA/edit?usp=sharing > > [2]: > > https://lists.apache.org/thread.html/3539984493cf3d4d439bef25c150fa9e09e0b43ce0afb6be378d41df@%3Cdev.arrow.apache.org%3E > > [3]: https://github.com/apache/arrow/pull/4980 > >
