BryanCutler commented on a change in pull request #12636:
URL: https://github.com/apache/arrow/pull/12636#discussion_r829505007
##########
File path: format/Flight.proto
##########
@@ -259,8 +259,10 @@ message FlightInfo {
FlightDescriptor flight_descriptor = 2;
/*
- * A list of endpoints associated with the flight. To consume the whole
- * flight, all endpoints must be consumed.
+ * A list of endpoints associated with the flight. To consume the
+ * whole flight, all endpoints (and hence all Tickets) must be consumed.
+ *
+ * In other words, multiple endpoints provide partitioning.
Review comment:
Sorry to come back to this, but I've had people come to me thinking that
Flight will do the partitioning for them. So we don't want to make it seem like
that is what's happening. Maybe reword slightly to something like "In other
words, multiple endpoints can be used with partitioned data." or just "...
multiple endpoints provide for partitioned data." ?
--
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]
