7146162: javax/management/remote/mandatory/connection/BrokenConnectionTest.java failing intermittently
Summary: ClientCommunicatorAdmin should call gotIOException((IOException)e) instead of restart((IOException)e) when detecting a communication error, because the method gotIOException will send a failure notification if necessary.
Reviewed-by: emcmanus, sjiang
Contributed-by: jaroslav.bachorik@oracle.com
--- a/jdk/src/share/classes/com/sun/jmx/remote/internal/ClientCommunicatorAdmin.java Thu Dec 20 20:11:45 2012 -0800
+++ b/jdk/src/share/classes/com/sun/jmx/remote/internal/ClientCommunicatorAdmin.java Fri Dec 21 09:27:37 2012 +0100
@@ -196,7 +196,7 @@
if (e instanceof IOException &&
!(e instanceof InterruptedIOException)) {
try {
- restart((IOException)e);
+ gotIOException((IOException)e);
} catch (Exception ee) {
logger.warning("Checker-run",
"Failed to check connection: "+ e);