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) {
