You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by ul...@apache.org on 2008/08/11 11:09:24 UTC
svn commit: r684687 -
/cxf/trunk/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSConduit.java
Author: ulhasbhole
Date: Mon Aug 11 02:09:23 2008
New Revision: 684687
URL: http://svn.apache.org/viewvc?rev=684687&view=rev
Log:
* Moved pooledSession recycle call to finally block for recycling the session in case of exceptions or normal return.
Modified:
cxf/trunk/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSConduit.java
Modified: cxf/trunk/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSConduit.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSConduit.java?rev=684687&r1=684686&r2=684687&view=diff
==============================================================================
--- cxf/trunk/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSConduit.java (original)
+++ cxf/trunk/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSConduit.java Mon Aug 11 02:09:23 2008
@@ -257,10 +257,11 @@
if (!isOneWay) {
handleResponse();
}
- base.sessionFactory.recycle(pooledSession);
} catch (JMSException jmsex) {
getLogger().log(Level.WARNING, "JMS connect failed with JMSException : ", jmsex);
throw new IOException(jmsex.toString());
+ } finally {
+ base.sessionFactory.recycle(pooledSession);
}
}