avantgardnerio opened a new issue, #2210:
URL: https://github.com/apache/arrow-rs/issues/2210

   **Is your feature request related to a problem or challenge? Please describe 
what you are trying to do.**
   
   Apache Arrow Ballista now supports FlightSQL, and can run simple statements 
like `select 1;`, however due to its architecture, tables need to be registered 
before any substantive queries can be run. Unfortunately, tables have to be 
registered on a context, or else the next query will have no idea that they 
were registered. The native Ballista protocol uses a [custom 
field](https://github.com/apache/arrow-ballista/blob/6bd0f6a58d0a7bdaf75af2c2485d56d5a812bea3/ballista/rust/core/proto/ballista.proto#L668)
 to implement this behavior, but for FlightSQL, we should use its [native 
method](https://arrow.apache.org/docs/format/Flight.html#authentication). 
However the existing `FlightSqlService` trait forces this [to 
return](https://github.com/apache/arrow-rs/blob/bc493d92c2a032a95d92dab81642f05182f20d81/arrow-flight/src/sql/server.rs#L261)
 an `unimplemented` `Err`.
   
   **Describe the solution you'd like**
   
   It seems reasonable to me to extend the `FlightSqlService` trait proxying 
the raw method signature from the `FlighService` trait.
   
   **Describe alternatives you've considered**
   
   I don't know enough about Rust to envision another approach.
   


-- 
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