Github user franz1981 commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2483#discussion_r244470859
--- 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 --
It is possible to use a primitive version of the map ie using primitive
longs instead of boxed types
---
