You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2014/10/14 08:41:56 UTC
svn commit: r1631636 -
/tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java
Author: rmannibucau
Date: Tue Oct 14 06:41:55 2014
New Revision: 1631636
URL: http://svn.apache.org/r1631636
Log:
reformatting, patch surely messed it up
Modified:
tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java
Modified: tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java?rev=1631636&r1=1631635&r2=1631636&view=diff
==============================================================================
--- tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java (original)
+++ tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/config/RemoteServer.java Tue Oct 14 06:41:55 2014
@@ -143,9 +143,9 @@ public class RemoteServer {
waitFor(p);
} else {
p.destroy();
+ }
}
}
- }
public void start() {
start(Collections.<String>emptyList(), START, true);
@@ -336,9 +336,9 @@ public class RemoteServer {
// kill3UNIXDebug();
final ProcessBuilder pb = new ProcessBuilder(args)
- .redirectOutput(ProcessBuilder.Redirect.INHERIT)
- .redirectError(ProcessBuilder.Redirect.INHERIT)
- .directory(home.getAbsoluteFile());
+ .redirectOutput(ProcessBuilder.Redirect.INHERIT)
+ .redirectError(ProcessBuilder.Redirect.INHERIT)
+ .directory(home.getAbsoluteFile());
Process p = pb.start();
@@ -355,17 +355,17 @@ public class RemoteServer {
throw (RuntimeException) new OpenEJBRuntimeException("Cannot start the server. Exception: " + e.getClass().getName() + ": " + e.getMessage()).initCause(e);
}
- if (debug) {
- if (!connect(port, Integer.MAX_VALUE)) {
+ if (debug) {
+ if (!connect(port, Integer.MAX_VALUE)) {
destroy();
- throw new OpenEJBRuntimeException("Could not connect to server");
- }
- } else {
- if (!connect(port, tries)) {
+ throw new OpenEJBRuntimeException("Could not connect to server");
+ }
+ } else {
+ if (!connect(port, tries)) {
destroy();
- throw new OpenEJBRuntimeException("Could not connect to server");
- }
+ throw new OpenEJBRuntimeException("Could not connect to server");
}
+ }
} else {
if (verbose) {
@@ -487,8 +487,8 @@ public class RemoteServer {
notSent();
}
- return false;
- }
+ return false;
+ }
}
private void notSent() {
@@ -501,11 +501,11 @@ public class RemoteServer {
// Check TomEE was effectively shut down after getting the message
// There can be concurrent shutdown operations (catalina shutdown hook for instance),
// so we have to wait here since it is important to be synchronous in this method
- waitForServerShutdown();
+ waitForServerShutdown();
} else {
if (verbose) {
notSent();
- }
+ }
}
}
@@ -517,15 +517,15 @@ public class RemoteServer {
final boolean b = disconnect(portShutdown, tries);
- if (verbose) {
+ if (verbose) {
System.out.println();
- }
+ }
if (!b) {
//We need to know about this
System.out.println("SEVERE: Failed to shutdown TomEE running on port " + portStartup + " using shutdown port: " + portShutdown);
}
- }
+ }
/**
* Send the shutdown message to the running server
@@ -547,7 +547,7 @@ public class RemoteServer {
} catch (final Exception e) {
if (attempts > 0) {
try {
- Thread.sleep(1000);
+ Thread.sleep(1000);
} catch (final InterruptedException ie) {
return false;
}