accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-845) Handle method.include calls consistently in TraceWrap class.
Date Fri, 16 Nov 2012 15:12:12 GMT

    [ https://issues.apache.org/jira/browse/ACCUMULO-845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13498849#comment-13498849
] 

Hudson commented on ACCUMULO-845:
---------------------------------

Integrated in Accumulo-Trunk #555 (See [https://builds.apache.org/job/Accumulo-Trunk/555/])
    ACCUMULO-845 make error handling consistent in the Proxy (Revision 1410378)

     Result = SUCCESS
ecn : 
Files : 
* /accumulo/trunk/trace/src/main/java/org/apache/accumulo/cloudtrace/instrument/thrift/TraceWrap.java

                
> Handle method.include calls consistently in TraceWrap class.
> ------------------------------------------------------------
>
>                 Key: ACCUMULO-845
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-845
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: David Medinets
>            Assignee: Eric Newton
>            Priority: Minor
>             Fix For: 1.5.0
>
>
> One call ignores exceptions and one call handles exceptions.
> {noformat}
>  if (args == null || args.length < 1 || args[0] == null || !(args[0] instanceof TInfo))
{
>   return method.invoke(instance, args);
>  }       
>  Span span = Trace.trace((TInfo) args[0], method.getName());
>  try {   
>     return method.invoke(instance, args);
>  } catch (InvocationTargetException ex) {
>    throw ex.getCause();
>  } finally {
>    span.stop();
>  }       
> {noformat}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message