Issue Details (XML | Word | Printable)

Key: AMQ-1179
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Hiram Chirino
Reporter: Hiram Chirino
Votes: 0
Watchers: 0
Operations

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

Broker Connection Shutdown is prone to lockup/deadlock

Created: 26/Feb/07 11:13 AM   Updated: 05/Mar/07 09:04 AM
Return to search
Component/s: Broker
Affects Version/s: 4.0, 4.0.1, 4.0.2, 4.1.0
Fix Version/s: 4.1.1, 5.0.0

Time Tracking:
Not Specified

Issue Links:
Related
 


 Description  « Hide
There are several scenarios (timing related) where the broker does not properly shutdown it's connections to a client.

 All   Comments   Work Log   Change History   Subversion Commits   FishEye   Crucible      Sort Order: Ascending order - Click to sort in descending order
Hiram Chirino added a comment - 26/Feb/07 11:22 AM
Several fixes were recently done in the 4.1 branch to make connection shutdown more robust:

Send the ShutdownInfo message in a async thread so that the operation can timeout if the connection fails while doing that:
http://svn.apache.org/viewvc?view=rev&rev=511090
http://svn.apache.org/viewvc?view=rev&rev=511091
http://svn.apache.org/viewvc?view=rev&rev=511089
http://svn.apache.org/viewvc?view=rev&rev=511083


Hiram Chirino made changes - 26/Feb/07 11:23 AM
Field Original Value New Value
Link This issue is related to AMQ-1052 [ AMQ-1052 ]
Hiram Chirino added a comment - 26/Feb/07 11:23 AM
AMQ-1052 was also cause problem properly shutting down a connection

Hiram Chirino added a comment - 26/Feb/07 11:23 AM
Need to port all the previously posted revisions to trunk.

Repository Revision Date User Message
AMQ #514734 Mon Mar 05 09:03:36 PST 2007 jlim ported fix to trunk :
http://issues.apache.org/activemq/browse/AMQ-1179
http://issues.apache.org/activemq/browse/AMQ-1180
Files Changed
MODIFY /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/TransportConnection.java
MODIFY /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Queue.java
MODIFY /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/Topic.java
MODIFY /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/memory/UsageManager.java
MODIFY /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/ConnectionContext.java
MODIFY /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/broker/region/TopicSubscription.java

Jonas Lim added a comment - 05/Mar/07 09:04 AM
ported fix to trunk : r514734

Jonas Lim made changes - 05/Mar/07 09:04 AM
Resolution Fixed [ 1 ]
Status Open [ 1 ] Resolved [ 5 ]
Create crucible review for this changeset in