You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ec...@apache.org on 2012/11/16 15:50:18 UTC
svn commit: r1410378 -
/accumulo/trunk/trace/src/main/java/org/apache/accumulo/cloudtrace/instrument/thrift/TraceWrap.java
Author: ecn
Date: Fri Nov 16 14:50:17 2012
New Revision: 1410378
URL: http://svn.apache.org/viewvc?rev=1410378&view=rev
Log:
ACCUMULO-845 make error handling consistent in the Proxy
Modified:
accumulo/trunk/trace/src/main/java/org/apache/accumulo/cloudtrace/instrument/thrift/TraceWrap.java
Modified: accumulo/trunk/trace/src/main/java/org/apache/accumulo/cloudtrace/instrument/thrift/TraceWrap.java
URL: http://svn.apache.org/viewvc/accumulo/trunk/trace/src/main/java/org/apache/accumulo/cloudtrace/instrument/thrift/TraceWrap.java?rev=1410378&r1=1410377&r2=1410378&view=diff
==============================================================================
--- accumulo/trunk/trace/src/main/java/org/apache/accumulo/cloudtrace/instrument/thrift/TraceWrap.java (original)
+++ accumulo/trunk/trace/src/main/java/org/apache/accumulo/cloudtrace/instrument/thrift/TraceWrap.java Fri Nov 16 14:50:17 2012
@@ -52,7 +52,11 @@ public class TraceWrap {
@Override
public Object invoke(Object obj, Method method, Object[] args) throws Throwable {
if (args == null || args.length < 1 || args[0] == null || !(args[0] instanceof TInfo)) {
- return method.invoke(instance, args);
+ try {
+ return method.invoke(instance, args);
+ } catch (InvocationTargetException ex) {
+ throw ex.getCause();
+ }
}
Span span = Trace.trace((TInfo) args[0], method.getName());
try {