liukun4515 commented on issue #2445:
URL: https://github.com/apache/arrow-rs/issues/2445#issuecomment-1225142558

   > @liukun4515 That seems out of date, the current definition in 
`Flight.proto` for Schema Result shows:
   > 
   > ```proto
   > /*
   >  * Wrap the result of a getSchema call
   >  */
   > message SchemaResult {
   >   // The schema of the dataset in its IPC form:
   >   //   4 bytes - an optional IPC_CONTINUATION_TOKEN prefix
   >   //   4 bytes - the byte length of the payload
   >   //   a flatbuffer Message whose header is the Schema
   >   bytes schema = 1;
   > }
   > ```
   > 
   > Defining that it should be the IPC message
   
   Thanks @zeroshade 
   I think the rust `flight.proto` is too old.
   
   In the rust side, the definition is
   ```
   /*
    * Wrap the result of a getSchema call
    */
   message SchemaResult {
     // schema of the dataset as described in Schema.fbs::Schema.
     bytes schema = 1;
   }
   ```
   
   @alamb  Maybe we should update the `.proto` in the rust side.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to