activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From franz1981 <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #1638: ARTEMIS-1499 ArtemisExecutor and Proces...
Date Fri, 03 Nov 2017 14:41:06 GMT
Github user franz1981 commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1638#discussion_r148800347
  
    --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/utils/actors/ProcessorBase.java
---
    @@ -109,12 +125,49 @@ public final boolean isFlushed() {
           return stateUpdater.get(this) == STATE_NOT_RUNNING;
        }
     
    -   protected void task(T command) {
    -      tasks.add(command);
    --- End diff --
    
    It is not only the flush that has issues but the execute too: take a look at the tests
I've added :)


---

Mime
View raw message