INTERNAL: Remove volatile keyword and change cause format.

naver · Aug 4, 2023 · c1b9b96 · c1b9b96
1 parent e487e3e
commit c1b9b96
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/main/java/net/spy/memcached/protocol/BaseOperationImpl.java b/src/main/java/net/spy/memcached/protocol/BaseOperationImpl.java
@@ -46,7 +46,7 @@ public abstract class BaseOperationImpl extends SpyObject {
    */
   public static final OperationStatus CANCELLED =
           new CancelledOperationStatus();
-  private volatile OperationState state = OperationState.WRITE_QUEUED;
+  private OperationState state = OperationState.WRITE_QUEUED;
   private ByteBuffer cmd = null;
   private boolean cancelled = false;
   private final AtomicBoolean callbacked = new AtomicBoolean(false);
@@ -96,10 +96,9 @@ public final boolean cancel(String cause) {
     if (callbacked.compareAndSet(false, true)) {
       cancelled = true;
       if (handlingNode != null) {
-        cancelCause = "Cancelled (" + cause + " : (" + handlingNode.getNodeName() + ")" + ")";
-      } else {
-        cancelCause = "Cancelled (" + cause + ")";
+        cause += " @ " + handlingNode.getNodeName();
       }
+      cancelCause = "Cancelled (" + cause + ")";
       wasCancelled();
       callback.complete();
       return true;