[GitHub] activemq-artemis pull request #2371: ARTEMIS-2097 Pause and Block Producers

[clebertsuconic]

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

    https://github.com/apache/activemq-artemis/pull/2371#discussion_r225357936
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ServerSessionImpl.java
 ---
    @@ -1789,6 +1790,11 @@ public synchronized RoutingStatus doSend(final 
Transaction tx,
     
           AddressInfo art = getAddressAndRoutingType(new 
AddressInfo(msg.getAddressSimpleString(), routingType));
     
    +      if (postOffice.isAddressBlocked(msg.getAddressSimpleString())) {
    --- End diff --
    
    We should have discussed this before you coming into implementation. 
    
    I don't think this is the right design.


---