From 88ca352ffee7df7e037bed1e73936e9b5d9b9ae2 Mon Sep 17 00:00:00 2001 From: Keith Wall Date: Fri, 16 Aug 2013 11:54:02 +0000 Subject: QPID-5050: Move invocation of ExceptionListener to after the failoverMutex is released avoiding deadlock possibility Previously, the ExceptionListener was invoked whilst the failoverMutex was held, between the two potential state changes (connection state change and session state change). This commit reorders the statements so that the ExceptionListner is fired after the failoverMutex is released. It also means that the ExceptionListener is fired *after* both connection/session have undergone any state changes. The exceptionListener member is also made thread safe. git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1514664 13f79535-47bb-0310-9956-ffa450edef68 --- qpid/java/test-profiles/Java010Excludes | 1 + 1 file changed, 1 insertion(+) (limited to 'qpid/java/test-profiles/Java010Excludes') diff --git a/qpid/java/test-profiles/Java010Excludes b/qpid/java/test-profiles/Java010Excludes index 055ce25d75..02c2db1134 100755 --- a/qpid/java/test-profiles/Java010Excludes +++ b/qpid/java/test-profiles/Java010Excludes @@ -20,6 +20,7 @@ // Those tests are testing 0.8..-0-9-1 specific semantics org.apache.qpid.test.client.ImmediateAndMandatoryPublishingTest#* org.apache.qpid.test.client.CloseOnNoRouteForMandatoryMessageTest#* +org.apache.qpid.test.unit.client.connection.ExceptionListenerTest#testExceptionListenerConnectionStopDeadlock org.apache.qpid.systest.rest.BrokerRestTest#testSetCloseOnNoRoute //this test checks explicitly for 0-8 flow control semantics -- cgit v1.2.1