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 Mon, 29 Oct 2018 17:00:09 GMT
Github user franz1981 commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2401#discussion_r229013748
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingStoreImpl.java
---
    @@ -700,7 +720,7 @@ public boolean checkMemory(final Runnable runWhenAvailable) {
     
        @Override
        public void addSize(final int size) {
    -      boolean globalFull = pagingManager.addSize(size).isGlobalFull();
    +      boolean globalFull = !ignoreGlobalMaxSize && pagingManager.addSize(size).isGlobalFull();
    --- End diff --
    
    @clebertsuconic @mtaylor 
    I'm not adding anything to the pagingManager to not trigger the other addresses policies:
it makes sense?


---

Mime
View raw message