You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@activemq.apache.org by CNNJYB <gi...@git.apache.org> on 2019/01/02 03:08:05 UTC

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

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

    https://github.com/apache/activemq-artemis/pull/2483#discussion_r244657469
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/AbstractJournalStorageManager.java ---
    @@ -193,7 +193,7 @@ public static JournalContent getType(byte type) {
     
        protected final Map<SimpleString, PersistedAddressSetting> mapPersistedAddressSettings = new ConcurrentHashMap<>();
     
    -   protected final Set<Long> largeMessagesToDelete = new HashSet<>();
    +   protected final Map<Long, LargeServerMessage> largeMessagesToDelete = new ConcurrentHashMap<>();
    --- End diff --
    
    @franz1981 modified, please review, Thanks.


---