indigophox commented on code in PR #34817:
URL: https://github.com/apache/arrow/pull/34817#discussion_r1399881605


##########
format/Flight.proto:
##########


Review Comment:
   Agreed as elsewhere this could/should be [in future] rolled into 
GetSessionOptions under an optional e.g. `supported_session_options` nested 
message, presumably as a map to optional vectors of valid values, with some 
decisions to be made around whether multiple value types are accommodates 
over-the-wire.



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