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

Masatake Iwasaki commented on HTRACE-119:
-----------------------------------------

After HTRACE-101, we need guard such as
{code}
ts = Trace.startSpan("MySpan");
if (ts != NullSpan.INSTANCE) {
  Span s = ts.detach();
  ...
}
{code}
or
{code}
if (Trace.isTracing()) {
  Span s = Trace.startSpan("MySpan").detach();
  ...
}
{code}
in order to avoid RuntimeException on detach.

> detach of NullScope singleton should not fail
> ---------------------------------------------
>
>                 Key: HTRACE-119
>                 URL: https://issues.apache.org/jira/browse/HTRACE-119
>             Project: HTrace
>          Issue Type: Bug
>    Affects Versions: master
>            Reporter: Masatake Iwasaki
>            Assignee: Masatake Iwasaki
>         Attachments: HTRACE-119.001.patch
>
>
> The {{detach}} method of NullScope singleton is possible to be called 
> multiple times by users avoiding conditionals by the assumption that 
> NullScope#close and NullScope#detach results in no-op. Calling {{detach}} of 
> NullScope singleton multiple times throws RuntimeException by HTRACE-101 now.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to