accumulo-notifications mailing list archives

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

    [ 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

Mime
View raw message