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 #2401: ARTEMIS-1710 Allow management msgs to p...
Date Wed, 31 Oct 2018 10:38:35 GMT
Github user franz1981 commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2401#discussion_r229640945
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingManagerImpl.java
---
    @@ -437,12 +449,27 @@ public void processReload() throws Exception {
           }
        }
     
    +   private boolean ignoreGlobalMaxSize(SimpleString address) {
    +      if (this.addressPrefixesIgnoringGlobalMaxSize == null) {
    +         return false;
    +      } else {
    +         //for a small number of prefixes to check we can use just a linear search too
:)
    +         for (SimpleString prefixes : this.addressPrefixesIgnoringGlobalMaxSize) {
    --- End diff --
    
    I taken the point and allowed that address Set to be just the one I need (if any), lefting
the code be simple as it needs to be.


---

Mime
View raw message