You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by el...@apache.org on 2014/12/09 19:15:39 UTC
[2/3] accumulo git commit: ACCUMULO-3395 Keep a reference to a single
Logger instance.
ACCUMULO-3395 Keep a reference to a single Logger instance.
Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/98c524e3
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/98c524e3
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/98c524e3
Branch: refs/heads/master
Commit: 98c524e3425678286872952ce376a4aa774079a5
Parents: 6170c02
Author: Josh Elser <el...@apache.org>
Authored: Tue Dec 9 13:11:03 2014 -0500
Committer: Josh Elser <el...@apache.org>
Committed: Tue Dec 9 13:11:03 2014 -0500
----------------------------------------------------------------------
.../main/java/org/apache/accumulo/server/util/RpcWrapper.java | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/accumulo/blob/98c524e3/server/base/src/main/java/org/apache/accumulo/server/util/RpcWrapper.java
----------------------------------------------------------------------
diff --git a/server/base/src/main/java/org/apache/accumulo/server/util/RpcWrapper.java b/server/base/src/main/java/org/apache/accumulo/server/util/RpcWrapper.java
index 02bd1ef..7055053 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/util/RpcWrapper.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/util/RpcWrapper.java
@@ -24,6 +24,7 @@ import org.apache.accumulo.trace.instrument.thrift.RpcServerInvocationHandler;
import org.apache.accumulo.trace.instrument.thrift.TraceWrap;
import org.apache.thrift.TApplicationException;
import org.apache.thrift.TException;
+import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
/**
@@ -38,17 +39,18 @@ public class RpcWrapper {
public static <T> T service(final T instance) {
InvocationHandler handler = new RpcServerInvocationHandler<T>(instance) {
+ private final Logger log = LoggerFactory.getLogger(instance.getClass());
@Override
public Object invoke(Object obj, Method method, Object[] args) throws Throwable {
try {
return super.invoke(obj, method, args);
} catch (RuntimeException e) {
String msg = e.getMessage();
- LoggerFactory.getLogger(instance.getClass()).error(msg, e);
+ log.error(msg, e);
throw new TException(msg);
} catch (Error e) {
String msg = e.getMessage();
- LoggerFactory.getLogger(instance.getClass()).error(msg, e);
+ log.error(msg, e);
throw new TException(msg);
}
}