[GitHub] curator pull request #242: [CURATOR-443] CuratorFrameworkImpl may sleep in f...

[Randgalt]

Github user Randgalt commented on a diff in the pull request:

    https://github.com/apache/curator/pull/242#discussion_r153336927
  
    --- Diff: 
curator-framework/src/main/java/org/apache/curator/framework/imps/CuratorFrameworkImpl.java
 ---
    @@ -973,6 +981,33 @@ void performBackgroundOperation(OperationAndData<?> 
operationAndData)
             }
         }
     
    +    @VisibleForTesting
    +    volatile long sleepAndQueueOperationSeconds = 1;
    --- End diff --
    
    I've been moving away from AtomicLong when compareAndSet ops aren't needed. 
Those wrappers ended up creating unnecessary garbage in the heap.


---