-
Notifications
You must be signed in to change notification settings - Fork 161
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you want to hide the span type "Local"? It is not useless.
This reverts commit a87dce5.
@@ -222,7 +222,12 @@ export default class TraceMap { | |||
.attr('text-anchor', 'start') | |||
.attr('fill', d => this.type[d.data.layer]) | |||
.attr('stroke', d => this.type[d.data.layer]) | |||
.text(d => d.data.layer); | |||
.text(d => { | |||
if(d.data.type == 'Local' && d.data.layer == 'Unknown'){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is that better?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can't just check unknown
, because some RPC plugin may forget to set layer.
I am merging this. Please submit UI sync pr. |
Issue Url: apache/skywalking#2200
Local span layer and layer text is Unknown => 'Local'
Snapshot like this: