History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: AMQ-1449
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Rob Davies
Reporter: Nicky Sandhu
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
ActiveMQ

Producer hangs on durable subscribe as Usage memory runs over

Created: 08/Oct/07 11:30 AM   Updated: 22/Oct/07 09:58 AM
Component/s: Broker
Affects Version/s: None
Fix Version/s: 5.0.0

Time Tracking:
Not Specified

File Attachments: 1. Text File tests.patch.txt (7 kb)


Regression: Broken Unit Test


 Description  « Hide
Broker hangs on a durable subscribe when producer is disconnected violently (System.exit()). I have included the test files to reproduce the problem

WARNING: The test cases included can be patched against activemq-core but are not unit tests. Do the following sequence

1.) Run the junit test DurableProducerConsumerBrokerTest. This starts a broker
2.) Run the junit test DurableProducerConsumerTest. It will exit with System.exit midway thru running the test
3.) Rerun the junit test DurableProducerConsumerTest...It should hang the second time around

Note: If you use the log4j.properties included in the patch you would see the Usage memory as it rises above 70 to 99+ in the second run.



 All   Comments   Work Log   Change History   Subversion Commits   FishEye   Crucible      Sort Order: Ascending order - Click to sort in descending order

Nicky Sandhu - 19/Oct/07 01:33 PM
TopicStorePrefetch has a system usage manager that is not used by Message (broker.region package). It instead uses the increment/decrement to inform the regionBroker's system usage manager which happens to be a different one.

So the listener implemented by DurableTopicSubscription never results in an actual call back and hence the leak ??

I don't understand this one completely but i hope it helps whoever looks at this issue


Rob Davies - 22/Oct/07 01:39 AM
Fixed by svn revision 587033

Nicky Sandhu - 22/Oct/07 09:58 AM
Thanks Rob! It does great under the performance tests as well. Any thoughts on how to include/modify these tests for the build ?