Skip to content

Instantly share code, notes, and snippets.

@maxandersen
Created March 8, 2012 11:13
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save maxandersen/2000505 to your computer and use it in GitHub Desktop.
Save maxandersen/2000505 to your computer and use it in GitHub Desktop.
diff --git a/as/plugins/org.jboss.ide.eclipse.as.management.as71/src/org/jboss/ide/eclipse/as/internal/management/as71/AS71Manager.java b/as/plugins/org.jboss.ide.eclipse.as.management.as71/src/org/jboss/ide/eclipse/as/internal/management/as71/AS71Manager.java
index ca24a6b..3e4abd7 100644
--- a/as/plugins/org.jboss.ide.eclipse.as.management.as71/src/org/jboss/ide/eclipse/as/internal/management/as71/AS71Manager.java
+++ b/as/plugins/org.jboss.ide.eclipse.as.management.as71/src/org/jboss/ide/eclipse/as/internal/management/as71/AS71Manager.java
@@ -31,6 +31,7 @@ import java.net.UnknownHostException;
import java.util.HashSet;
import java.util.List;
import java.util.Set;
+import java.util.concurrent.ExecutionException;
import java.util.concurrent.Future;
import javax.security.auth.callback.Callback;
@@ -288,6 +289,7 @@ public class AS71Manager {
try {
client.execute(node);
} catch (Exception e) {
+ // AS 7.0 servers fails in finishing the client calls thus we will see IOException when calling shutdown that we should ignore.
if (!isConnectionCloseException(e)) {
throw new JBoss7ManangerException(e);
}
@@ -295,9 +297,10 @@ public class AS71Manager {
}
private boolean isConnectionCloseException(Exception e) {
- return e instanceof IOException
+ return (e instanceof IOException
&& e.getMessage() != null
- && e.getMessage().indexOf("Channel closed") > -1;
+ && (e.getMessage().indexOf("Channel closed") > -1) // pre-AS 7.1 client lib
+ || e.getMessage().indexOf("Operation failed") > -1); // AS 7.1 client lib
}
private IJBoss7DeploymentResult execute(DeploymentPlanBuilder builder) throws JBoss7ManangerException {
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment