lidavidm commented on a change in pull request #12636:
URL: https://github.com/apache/arrow/pull/12636#discussion_r829554864
##########
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.
Review comment:
Updated.
Though - that brings up the question of how to represent an ordered dataset.
Is the expectation that (say) if results are sorted, then the endpoints should
also be sorted?
##########
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:
That's a very good point. I've reworded things here, how does it look
now?
--
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]
