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

    https://github.com/apache/activemq-artemis/pull/2175#discussion_r201392536
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java
 ---
    @@ -270,6 +271,15 @@
     
        private final QueueFactory factory;
     
    +   private final AtomicBoolean dispatching = new AtomicBoolean();
    --- End diff --
    
    @franz1981 so after going back to see if i could just use != -1, i 
remembered why i couldnt it would mean adding a System.currentMillis call on 
the hot path, which is very costly.
    
    I have though moved to using AtomicUpdaters , though :) 


---

Reply via email to