clebertsuconic commented on a change in pull request #2549: ARTEMIS-2251 Large 
messages might not be deleted when server crashed
URL: https://github.com/apache/activemq-artemis/pull/2549#discussion_r258697367
 
 

 ##########
 File path: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java
 ##########
 @@ -432,7 +432,7 @@ public void freeDirectBuffer(ByteBuffer buffer) {
       journalFF.releaseBuffer(buffer);
    }
 
-   public long storePendingLargeMessage(final long messageID, long recordID) 
throws Exception {
 
 Review comment:
   @qihongxu I fixed it without a blocking operation through a different PR. 
Please let me know if you see any issues.
   
   Thanks
   
   @michaelandrepearce  ^^

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to