Github user franz1981 commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2175#discussion_r201076355
--- 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 --
> Agree could just use != -1 check, memory vs cpu cycle
TBH to me are good enough both, but if we can just check !=-1 it would be
welcome, even if less readable...
I'm more worried about the writes cost then the reads :P
---
