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();
         }
     }
 


Reply via email to