Clarify cancel semantics

apache · Jul 5, 2022 · aba56c6 · aba56c6
1 parent 8326857
commit aba56c6
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/format/FlightSql.proto b/format/FlightSql.proto
@@ -1684,10 +1684,12 @@ message DoPutUpdateResult {
 /*
  * 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. The transaction/statement is not rolled back. This only indicates
- * the client no longer wishes to read the remainder of the query results.
+ * This lets a single client explicitly cancel work, no matter how many clients
+ * are involved/whether the query is distributed or not, given server
+ * support. The transaction/statement is not rolled back; it is the
+ * application's job to commit or rollback as appropriate. This only indicates
+ * the client no longer wishes to read the remainder of the query
+ * results or continue submitting data.
  */
 message ActionCancelQuery {
   // The result of the GetFlightInfo RPC that initated the query.