alamb commented on code in PR #36946:
URL: https://github.com/apache/arrow/pull/36946#discussion_r1286956357
##########
format/Flight.proto:
##########
@@ -343,6 +368,43 @@ message FlightInfo {
bool ordered = 6;
}
+/*
+ * The information to process a long-running query.
+ */
+message RetryInfo {
+ /*
+ * The currently available results so far.
+ *
+ * It's a complete FlightInfo each time not just the delta between
+ * the previous and current FlightInfo.
+ *
+ * A server should only append to the endpoints in the info. So the
+ * client can run DoGet(Ticket) with the Ticket in the info even
+ * when the query isn't finished yet. FlightInfo.ordered is also
+ * valid.
Review Comment:
```suggestion
* The currently available results.
*
* If `flight_descriptor` is not specified, the query is complete and
* `info` specifies all results. Otherwise, `info` contains partial query
results.
*
* Note that each `RetryInfo` response contains a complete FlightInfo
* (not just the delta between the previous and current FlightInfo).
*
* Subsequent `RetryInfo` responses may only append new endpoints to info.
*
* Clients can begin fetching results via DoGet(Ticket) with the ticket in
the info before
* the query is completed. FlightInfo.ordered is also valid.
```
--
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]
