[ 
https://issues.apache.org/jira/browse/AMQ-6014?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15021022#comment-15021022
 ] 

Christopher L. Shannon commented on AMQ-6014:
---------------------------------------------

[~gtully],

Looks like this fix is causing a test failure in this test: 
org.apache.activemq.usecases.DurableSubscriberNonPersistentMessageTest.testDurableSubscriberNonPersistentMessage

It seems that the issue is the extra call node.incrementReferenceCount() in the 
createMessageDispatch() method of DurableTopicSubscription is causing the 
cursorMemoryUsage value checked in the test to no longer be 0.  Not sure if 
this is an actual regression or if the failing test just needs to be adjusted 
for the new behavior that you added in this patch.

> Offline Durable Topic Subscription exceeds memory limits
> --------------------------------------------------------
>
>                 Key: AMQ-6014
>                 URL: https://issues.apache.org/jira/browse/AMQ-6014
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.12.0
>            Reporter: Gary Tully
>            Assignee: Gary Tully
>             Fix For: 5.13.0, 5.12.2
>
>
> To reproduce:
> 1) Start two durable subscriptions using the openwire swissarmy examples:
> ant consumer -Dtopic=true -Ddurable=true -Dsubject=test -Duser=admin 
> -Dpassword=admin -Dmax 500000
> ant consumer -Dtopic=true -Ddurable=true -Dsubject=test -Duser=admin 
> -Dpassword=admin -DclientId=c2
> 2) Start a producer using the following command:
> ant producer -Dtopic=true -Ddurable=true -Dsubject=test -Duser=admin 
> -Dpassword=admin -Dmax=500000
> 3) Kill one of the durable topic subscribers and let the test run
> 4) the broker will go OOM
> The cache for the offline durable sub is not recording is cached messages 
> against system memory usage as it should. Hence the limits are not hit.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to