lidavidm commented on code in PR #13492:
URL: https://github.com/apache/arrow/pull/13492#discussion_r921104977
##########
format/FlightSql.proto:
##########
@@ -1549,6 +1658,17 @@ message DoPutUpdateResult {
int64 record_count = 1;
}
+/*
+ * Explicitly cancel a running query.
+ *
+ * For distributed queries (ones that use GetFlightInfo->DoGet), this lets a
+ * single client explicitly cancel work across all clients, given server
support.
+ */
+message ActionCancelQuery {
+ // The result of the GetFlightInfo RPC that initated the query.
+ FlightInfo info = 1;
Review Comment:
Yes - if we had FlightInfo.app_metadata, we could return an explicit
cancellation token. But given the server is generating the tickets, it should
be OK for the server to also introspect them, so long as the client doesn't.
--
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]
