Github user michaelandrepearce commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1753
  
    @franz1981 LGTM. 
    
    The only comment i would have, is it worth adding the code to handle the 
offheap now also, just while its fresh in your mind, and then it be more 
complete feature. And if you disagree then thats also fine, id be still happy 
for this to merge.


---

Reply via email to