[GitHub] [arrow] zeroshade commented on issue #37635: [FlightRPC] FlightSQL: Expose custom query metadata

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

zeroshade commented on issue #37635:
URL: https://github.com/apache/arrow/issues/37635#issuecomment-1712117941

   I can't speak for the Java implementation, but many FlightRPC implementations expose the `app_metadata` field on the `FlightData` messages during a stream which tends to be the common method for including this type of metadata (the other method tends to be including it in the schema of the result set, i.e. the "hacky" way you mentioned). Personally I prefer it being part of the `FlightData` as it ensures that every message is able to contain metadata which may potentially include things like `partitionId` and so on above and beyond the query level metadata.
   
   @kou @lidavidm what do you think?


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