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

Colin Patrick McCabe commented on HTRACE-148:
---------------------------------------------

Thanks again, [~iwasakims].  It's frustrating that this problem only seems to 
appear for you (I've never seen it.)

Can you make the {{MilliSpan#fromJson}} method package-private instead of 
public?  We'd have to think about it more if we wanted a fully public 
interface, I think.  Deserializing from a string is less efficient than some of 
the other interfaces we could have.  Since this function is just called by unit 
tests in the same package, I guess we don't need to make it public.

Looks good aside from that.

> TestHTracedRESTReceiver should use the shaded ObjectMapper to avoid 
> TestHTracedRESTReceiver failure
> ---------------------------------------------------------------------------------------------------
>
>                 Key: HTRACE-148
>                 URL: https://issues.apache.org/jira/browse/HTRACE-148
>             Project: HTrace
>          Issue Type: Bug
>            Reporter: Masatake Iwasaki
>            Assignee: Masatake Iwasaki
>             Fix For: 3.2.0
>
>         Attachments: HTRACE-148.001.patch, HTRACE-148.002.patch, 
> HTRACE-148.003.patch
>
>




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

Reply via email to