Bug 495655 - cluster does not free up sessions on failover
Summary: cluster does not free up sessions on failover
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Red Hat Enterprise MRG
Classification: Red Hat
Component: qpid-cpp
Version: Development
Hardware: All
OS: Linux
urgent
urgent
Target Milestone: 1.1.1
: ---
Assignee: messaging-bugs
QA Contact: MRG Quality Engineering
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-04-14 09:31 UTC by Gordon Sim
Modified: 2011-06-29 14:16 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-06-29 14:16:27 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
test case (1.51 KB, text/x-c++src)
2009-04-14 09:31 UTC, Gordon Sim
no flags Details

Description Gordon Sim 2009-04-14 09:31:03 UTC
Created attachment 339448 [details]
test case

Description of problem:

If you use a session with an application specified name, then on failover to another cluster node try to open a new session with the same name, you get a SessionBusyException. 

Even retrying after a short wait seems not to avoid this.

How reproducible:

100%

Steps to Reproduce:
1. start cluster with more than one node
2. have client that creates a session with a given name (and 0 timeout, which is default) and e.g. waits for a message 
3. kill the node the client is connected to
4. have client failover to other node and create a new session with the same name as that used previously (which should have been destroyed)

The attached case provides such a client. You should be able to run that against  a cluster (with a queue called test-queue created), then kill the node connected to.
  
Actual results:

Get a SessionBusyException as often as you try.

Expected results:

New session created with same name as that used previously, which should now have been destroyed.

Additional info:

If the session declares an exclusive (but not auto-deleted) queue, then it appears that lock on that queue is released as would be expected. However the name of the session is still marked as busy.

Comment 1 Alan Conway 2009-04-14 14:20:25 UTC
This was a regression. Fixed in r764783

Comment 2 Justin Ross 2011-06-28 19:30:38 UTC
Does this have test coverage?

Comment 3 Alan Conway 2011-06-29 13:27:25 UTC
No automated tests were added by r764783. I'll add one now.

Comment 4 Alan Conway 2011-06-29 14:11:22 UTC
Comment 3 is incorrect, there IS an automated regression test added by r764783:
 qpid/cpp/src/tests/ClusterFailover.cpp

Comment 5 Justin Ross 2011-06-29 14:16:27 UTC
Excellent, thanks!  Closing.


Note You need to log in before you can comment on or make changes to this bug.