activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From CNNJYB <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...
Date Wed, 02 Jan 2019 03:08:05 GMT
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.


---

Mime
View raw message