tree ded656e049bc4a9ed51517a202195b4c812738ed
parent 62abcc1da271c3879d3454c54e160d8c52152e85
author Ian Cottrell <iancottrell@google.com> 1584707388 -0400
committer Ian Cottrell <iancottrell@google.com> 1585098044 +0000

internal/telemetry: replace TagSet with TagMap and TagPointer

This separates the concerns of tag collections that have to be iterated
and tag collections that need lookup by key.
Also make it so that events just carry a plain slice of tags.
We pass a TagMap down through the exporters and allow it to be
extended on the way.
We no longer need the event.Query method (or the event type)
We now exclusivley use Key as the identity, and no longer have a
common core implementation but just implement it directly in each
type.
This removes some confusion that was causing the same key through
different paths to end up with a different identity.

Change-Id: I61e47adcb397f4ca83dd90342b021dd8e9571ed3
Reviewed-on: https://go-review.googlesource.com/c/tools/+/224278
Run-TryBot: Ian Cottrell <iancottrell@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com>