indigophox commented on code in PR #34817:
URL: https://github.com/apache/arrow/pull/34817#discussion_r1394773842
##########
format/Flight.proto:
##########
@@ -525,3 +525,108 @@ message FlightData {
message PutResult {
bytes app_metadata = 1;
}
+
+/*
+ * EXPERIMENTAL: Union of possible value types for a Session Option to be set
to.
+ */
+message SessionOptionValue {
Review Comment:
Yes. We very explicitly wanted more int/etc values for programmatic use
cases of passing exact numeric parameters in their expected type container. I
had also considered that Value gets less interesting with Java and signed ints,
but nonetheless as with the strict map (rather than loose k:v pairs) we wanted
the constraint of the values coming in in-bounds so a server implementation
doesn't need to boundscheck them and care about informative errors and so on.
--
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]
