[
https://issues.apache.org/jira/browse/HTRACE-111?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14329496#comment-14329496
]
Colin Patrick McCabe commented on HTRACE-111:
---------------------------------------------
bq. nit to fix on commit: if (span.getParents().length == 0) { should be
isEmpty?
thx
bq. Q: If no begin, do we not default this: begin(System.currentTimeMillis() ?
We currently default to 0. I guess we could map no setting to
System.currentTimeMillis... let's think about it if we have to add more uses of
this.
bq. I like this. This is a better way of saying 'no parent': if
(span.getParents().length == 0) { Do you think it worth adding a 'hasParent'
convenience method?
Let's keep the interface minimal for now. It's so hard to remove methods once
you add them. We should keep it in mind for the future, though.
> HTrace Java client API fixes for 3.2
> ------------------------------------
>
> Key: HTRACE-111
> URL: https://issues.apache.org/jira/browse/HTRACE-111
> Project: HTrace
> Issue Type: Bug
> Affects Versions: 3.2.0
> Reporter: Colin Patrick McCabe
> Assignee: Colin Patrick McCabe
> Priority: Critical
> Attachments: HTRACE-111.001.patch
>
>
> * Remove old MilliSpan constructors in favor of {{MilliSpan#Builder}}
> * Restore {{Trace#startSpan(String, TraceInfo)}} because it's used in Hadoop
> * Remove {{Span#getParentId}} API which assumes a single parent world, as
> well as {{ROOT_SPAN_ID}}.
> * Use regular Random instead of SecureRandom.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
