User: starksm
Date: 02/02/28 00:08:39
Modified: src/main/org/jboss/invocation/jrmp/server JRMPInvoker.java
Log:
Do not log invocation exceptions unless trace is enabled as this
will cause normal application exceptions to be logged as errors.
Revision Changes Path
1.16 +9 -6 jboss/src/main/org/jboss/invocation/jrmp/server/JRMPInvoker.java
Index: JRMPInvoker.java
===================================================================
RCS file:
/cvsroot/jboss/jboss/src/main/org/jboss/invocation/jrmp/server/JRMPInvoker.java,v
retrieving revision 1.15
retrieving revision 1.16
diff -u -r1.15 -r1.16
--- JRMPInvoker.java 27 Feb 2002 05:40:20 -0000 1.15
+++ JRMPInvoker.java 28 Feb 2002 08:08:39 -0000 1.16
@@ -56,7 +56,7 @@
* from RMI/JRMP into the JMX base.
*
* @author <a href="mailto:[EMAIL PROTECTED]>Marc Fleury</a>
- * @version $Revision: 1.15 $
+ * @version $Revision: 1.16 $
*
* <p><b>Revisions:</b><br>
* <p><b>2002/01/13: Sacha Labourey</b>
@@ -377,15 +377,18 @@
if (e instanceof RuntimeOperationsException)
e = ((RuntimeOperationsException)e).getTargetException();
-
- log.error("operation failed", e);
- throw e;
+
+ // Only log errors if trace is enabled
+ if( log.isTraceEnabled() )
+ log.trace("operation failed", e);
+ throw e;
}
- finally {
+ finally
+ {
Thread.currentThread().setContextClassLoader(oldCl);
}
}
-
+
// Package protected ---------------------------------------------
// Protected -----------------------------------------------------
_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development
