You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by bh...@apache.org on 2007/02/15 15:25:50 UTC
svn commit: r507938 -
/incubator/qpid/trunk/qpid/java/perftests/src/test/java/org/apache/qpid/ping/PingTestPerf.java
Author: bhupendrab
Date: Thu Feb 15 06:25:49 2007
New Revision: 507938
URL: http://svn.apache.org/viewvc?view=rev&rev=507938
Log:
If there is any error in closing the connection, then also the thread setup should be removed
Modified:
incubator/qpid/trunk/qpid/java/perftests/src/test/java/org/apache/qpid/ping/PingTestPerf.java
Modified: incubator/qpid/trunk/qpid/java/perftests/src/test/java/org/apache/qpid/ping/PingTestPerf.java
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/perftests/src/test/java/org/apache/qpid/ping/PingTestPerf.java?view=diff&rev=507938&r1=507937&r2=507938
==============================================================================
--- incubator/qpid/trunk/qpid/java/perftests/src/test/java/org/apache/qpid/ping/PingTestPerf.java (original)
+++ incubator/qpid/trunk/qpid/java/perftests/src/test/java/org/apache/qpid/ping/PingTestPerf.java Thu Feb 15 06:25:49 2007
@@ -237,13 +237,15 @@
perThreadSetup._pingClient.close();
}
}
-
- // Ensure the per thread fixture is reclaimed.
- threadSetup.remove();
}
catch (JMSException e)
{
_logger.warn("There was an exception during per thread tear down.");
+ }
+ finally
+ {
+ // Ensure the per thread fixture is reclaimed.
+ threadSetup.remove();
}
}