git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] activemq-artemis pull request #2414: ARTEMIS-1710 Allow management msgs to e...


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

    https://github.com/apache/activemq-artemis/pull/2414#discussion_r230972615
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/AbstractJournalStorageManager.java ---
    @@ -1244,6 +1244,9 @@ private static PageSubscription locateSubscription(final long queueID,
              if (queueInfo != null) {
                 SimpleString address = queueInfo.getAddress();
                 PagingStore store = pagingManager.getPageStore(address);
    --- End diff --
    
    @mtaylor I'm considering to be compatible with old server. There maybe some old management address paging folders. When we upgrade,  getPageStore() may return null. That doesn't matter as we can delete old management folders and restart.


---