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 2012/09/21 17:01:45 UTC
svn commit: r1388542 -
/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/monitoring/remote/RemoteResourceMonitor.java
Author: rmannibucau
Date: Fri Sep 21 15:01:45 2012
New Revision: 1388542
URL: http://svn.apache.org/viewvc?rev=1388542&view=rev
Log:
better message for ping MBean
Modified:
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/monitoring/remote/RemoteResourceMonitor.java
Modified: openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/monitoring/remote/RemoteResourceMonitor.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/monitoring/remote/RemoteResourceMonitor.java?rev=1388542&r1=1388541&r2=1388542&view=diff
==============================================================================
--- openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/monitoring/remote/RemoteResourceMonitor.java (original)
+++ openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/monitoring/remote/RemoteResourceMonitor.java Fri Sep 21 15:01:45 2012
@@ -72,7 +72,7 @@ public class RemoteResourceMonitor imple
}
final ObjectNameBuilder jmxName = new ObjectNameBuilder("openejb.management");
- jmxName.set("ObjectType", "hosts");
+ jmxName.set("ObjectType", "Related Hosts");
objectName = jmxName.build();
try {
@@ -145,13 +145,18 @@ public class RemoteResourceMonitor imple
private static String ping(final String host) {
try {
final InetAddress address = InetAddress.getByName(host);
- final long start = System.nanoTime();
boolean ok = address.isReachable(30000);
- final long end = System.nanoTime();
- if (!ok) {
- return "Can't ping host, timeout (30s)";
+ if (ok) { // do it twice since the first one is generally longer
+ final long start = System.nanoTime();
+ ok = address.isReachable(30000);
+ final long end = System.nanoTime();
+ if (ok) {
+ final long duration = end - start;
+ final long ms = TimeUnit.NANOSECONDS.toMillis(duration);
+ return "Ping done in " + ms + "." + Long.toString(duration - 1000 * ms) + " ms";
+ }
}
- return Long.toString(TimeUnit.NANOSECONDS.toMillis(end - start));
+ return "Can't ping host, timeout (30s)";
} catch (UnknownHostException e) {
return "Can't find host: " + e.getMessage();
} catch (IOException e) {