Github user gaohoward commented on the pull request:

    https://github.com/apache/activemq-6/pull/211#issuecomment-95559754
  
    Ok although the unit test pass, but with manual test I sometimes got OOM 
still. Look at this code in ChannelImpl.send():
    
    resendCache.add(packet);
    
    
https://github.com/clebertsuconic/activemq-6/blob/new-Bridge/activemq-core-client/src/main/java/org/apache/activemq/core/protocol/core/impl/ChannelImpl.java#L294
    
    When sending a large, each packet holds a chunk of it, and because it's 
stored in the resendCache, it doesn't get gc'ed after the bytes are sent to the 
other node. Eventually before the large send complete, all previous sent chunks 
will be held in memory. 
    
    any comment?
    
    Howard



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to