Explicitly make cancellation idempotent

apache · Jul 26, 2022 · eae9ee1 · eae9ee1
1 parent 1024b56
commit eae9ee1
Showing 1 changed file with 17 additions and 1 deletion.
diff --git a/format/FlightSql.proto b/format/FlightSql.proto
@@ -1700,12 +1700,28 @@ message DoPutUpdateResult {
  * 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 ActionCancelQuery {
+message ActionCancelQueryRequest {
   // The result of the GetFlightInfo RPC that initated the query.
   FlightInfo info = 1;
 }
 
+/*
+ * The result of cancelling a query.
+ */
+message ActionCancelQueryResult {
+  enum CancelQueryResult {
+    CANCEL_QUERY_RESULT_UNSPECIFIED = 0;
+    CANCEL_QUERY_RESULT_CANCELLED = 1;
+    CANCEL_QUERY_RESULT_CANCELLING = 2;
+    CANCEL_QUERY_RESULT_NOT_CANCELLABLE = 3;
+  }
+
+  CancelQueryResult result = 1;
+}
+
 extend google.protobuf.MessageOptions {
   bool experimental = 1000;
 }