zeroshade commented on code in PR #13492:
URL: https://github.com/apache/arrow/pull/13492#discussion_r953857778


##########
format/FlightSql.proto:
##########
@@ -1550,6 +1755,48 @@ message DoPutUpdateResult {
   int64 record_count = 1;
 }
 
+/*
+ * Request message for the "CancelQuery" action.
+ *
+ * Explicitly cancel a running query.
+ *
+ * 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.
+ *
+ * This command is idempotent.
+ */
+message ActionCancelQueryRequest {
+  option (experimental) = true;
+
+  // The result of the GetFlightInfo RPC that initated the query.
+  // XXX(ARROW-16902): this must be a serialized FlightInfo, but is
+  // rendered as bytes because Protobuf does not really support one
+  // DLL using Protobuf definitions from another DLL.
+  bytes info = 1;
+}
+
+/*
+ * The result of cancelling a query.
+ *
+ * The result should be wrapped in a google.protobuf.Any message.
+ */
+message ActionCancelQueryResult {
+  option (experimental) = true;
+
+  enum CancelResult {
+    CANCEL_RESULT_UNSPECIFIED = 0;
+    CANCEL_RESULT_CANCELLED = 1;
+    CANCEL_RESULT_CANCELLING = 2;
+    CANCEL_RESULT_NOT_CANCELLABLE = 3;

Review Comment:
   yup much better! thanks!



-- 
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]

Reply via email to