]> Cypherpunks repositories - gostls13.git/commit
cmd/trace: assign a unique span id for slice representation
authorHana Kim <hakim@google.com>
Tue, 27 Mar 2018 18:41:19 +0000 (14:41 -0400)
committerHyang-Ah Hana Kim <hyangah@gmail.com>
Tue, 27 Mar 2018 19:03:54 +0000 (19:03 +0000)
commitaaeaad687065f45efedf2e070780b8db6ceb7f17
tree0567fa935925222acfae616e4449d6856e478613
parent331c187b173e5128ed24a0670c3e5010498c1954
cmd/trace: assign a unique span id for slice representation

Spans are represented using Async Event types of chrome trace viewer.
According to the doc, the 'id' should be unique within category, scope.

https://docs.google.com/document/d/1CvAClvFfyA5R-PhYUmn5OOQtYMH4h6I0nSsKchNAySU/preview#heading=h.jh64i9l3vwa1

Use the index in the task's span slice as the slice id, so it
can be unique within the task. The scope is the task id which
is unique.

This fixes a visualization bug that caused incorrect or missing
presentation of nested spans.

Change-Id: If1537ee00247f71fa967abfe45569a9e7dbcdce7
Reviewed-on: https://go-review.googlesource.com/102697
Reviewed-by: Heschi Kreinick <heschi@google.com>
src/cmd/trace/trace.go