[GitHub] [accumulo] milleruntime commented on a change in pull request #2467: Added mechanism to cancel FaTE operations from Shell with Manager up

[GitBox <gi...@apache.org>]

milleruntime commented on a change in pull request #2467:
URL: https://github.com/apache/accumulo/pull/2467#discussion_r800648662



##########
File path: core/src/main/java/org/apache/accumulo/fate/Fate.java
##########
@@ -300,6 +328,56 @@ public TStatus waitForCompletion(long tid) {
     return store.waitForStatusChange(tid, FINISHED_STATES);
   }
 
+  /**
+   * Attempts to cancel a running Fate transaction
+   *
+   * @param tid
+   *          transaction id
+   * @return true if transaction transitioned to a failed state or already in a completed state,
+   *         false otherwise
+   */
+  public boolean cancel(long tid) {
+    String tidStr = Long.toHexString(tid);
+    for (int retries = 0; retries < 5; retries++) {
+      if (store.tryReserve(tid)) {
+        try {
+          TStatus status = store.getStatus(tid);
+          if (status == TStatus.NEW || status == TStatus.SUBMITTED
+              || status == TStatus.IN_PROGRESS) {
+            store.setProperty(tid, EXCEPTION_PROP, new TApplicationException(

Review comment:
       What is `EXCEPTION_PROP`? I can't seem to find it.




-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org