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

[franz1981]

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

    https://github.com/apache/activemq-artemis/pull/2483#discussion_r244470771
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java
 ---
    @@ -461,8 +462,7 @@ void deleteLargeMessageFile(final LargeServerMessage 
largeServerMessage) throws
              try {
                 if (isReplicated() && replicator.isSynchronizing()) {
                    synchronized (largeMessagesToDelete) {
    --- End diff --
    
    If it's now using CHM there is no need to sync on it


---