You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by ke...@apache.org on 2010/03/04 14:48:16 UTC
svn commit: r918990 -
/openejb/trunk/openejb3/server/openejb-client/src/main/java/org/apache/openejb/client/Client.java
Author: kevan
Date: Thu Mar 4 13:48:15 2010
New Revision: 918990
URL: http://svn.apache.org/viewvc?rev=918990&view=rev
Log:
OPENEJB-1239 Always discard a session if there is a failure. Otherwise, we may try to reuse an inoperable connection
Modified:
openejb/trunk/openejb3/server/openejb-client/src/main/java/org/apache/openejb/client/Client.java
Modified: openejb/trunk/openejb3/server/openejb-client/src/main/java/org/apache/openejb/client/Client.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb3/server/openejb-client/src/main/java/org/apache/openejb/client/Client.java?rev=918990&r1=918989&r2=918990&view=diff
==============================================================================
--- openejb/trunk/openejb3/server/openejb-client/src/main/java/org/apache/openejb/client/Client.java (original)
+++ openejb/trunk/openejb3/server/openejb-client/src/main/java/org/apache/openejb/client/Client.java Thu Mar 4 13:48:15 2010
@@ -252,10 +252,10 @@
} catch (RemoteException e) {
throw e;
} catch (IOException e){
+ Set<URI> failed = getFailed();
+ failed.add(conn.getURI());
+ conn.discard();
if (retry){
- Set<URI> failed = getFailed();
- failed.add(conn.getURI());
- conn.discard();
try {
processRequest(req, res, server);
} catch (RemoteFailoverException re) {