[GitHub] [arrow-rs] crepererum opened a new issue, #3808: `FlightSqlServiceClient`: response metadata

["crepererum (via GitHub)" <gi...@apache.org>]

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

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   The gRPC response metadata of some FlightSQL servers contains useful information like date/time, software versions, or trace IDs. Users may either use this directly or at least log it, e.g. in case of the [CLI client](https://github.com/apache/arrow-rs/pull/3789).
   
   **Describe the solution you'd like**
   It would be useful if the methods in `FlightSqlServiceClient` (e.g. `execute`) would return the metadata alongside the deserialized body (`FlightInfo` in case of `metadata`). This could either be part of the default methods (return a tuple) or we could have every method twice, e.g. `execute` and `execute_return_md` (naming to be decided).
   
   I personally would prefer NOT to double the method count.
   
   **Describe alternatives you've considered**
   Not implementing this because it's "too low level" and have people write their own client if they need this feature. The fact that the demo CLI client already could use this and the feature is missing however makes me think that this should be a builtin feature.
   
   **Additional context**
   \-
   


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-rs] tustvold commented on issue #3808: `FlightSqlServiceClient`: response metadata

["tustvold (via GitHub)" <gi...@apache.org>]

tustvold commented on issue #3808:
URL: https://github.com/apache/arrow-rs/issues/3808#issuecomment-1456277832

   Perhaps the methods could return something like a `FlightResponse<T>` that implements `Deref<T>` but also has a `metadata` method?


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org