[jira] [Commented] (ACCUMULO-845) Handle method.include calls consistently in TraceWrap class.

["Eric Newton (JIRA)" <ji...@apache.org>]

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

Eric Newton commented on ACCUMULO-845:
--------------------------------------

I'm going to fix this in trunk.  Do you think this should be back-ported to 1.4?
                
> 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
>            Priority: Minor
>
> 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