You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ri...@apache.org on 2007/02/14 11:00:58 UTC
svn commit: r507460 -
/incubator/qpid/branches/perftesting/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession.java
Author: ritchiem
Date: Wed Feb 14 02:00:57 2007
New Revision: 507460
URL: http://svn.apache.org/viewvc?view=rev&rev=507460
Log:
Reverted change in position of closing consumers.
Modified:
incubator/qpid/branches/perftesting/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession.java
Modified: incubator/qpid/branches/perftesting/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession.java
URL: http://svn.apache.org/viewvc/incubator/qpid/branches/perftesting/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession.java?view=diff&rev=507460&r1=507459&r2=507460
==============================================================================
--- incubator/qpid/branches/perftesting/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession.java (original)
+++ incubator/qpid/branches/perftesting/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession.java Wed Feb 14 02:00:57 2007
@@ -656,6 +656,9 @@
if (!_closed.getAndSet(true))
{
+ // we pass null since this is not an error case
+ closeProducersAndConsumers(null);
+
try
{
_connection.getProtocolHandler().closeSession(this);
@@ -672,8 +675,6 @@
// When control resumes at this point, a reply will have been received that
// indicates the broker has closed the channel successfully
- // we pass null since this is not an error case
- closeProducersAndConsumers(null);
}
catch (AMQException e)