-
Notifications
You must be signed in to change notification settings - Fork 97
Tags: Add support for TagMetadata #424
Tags: Add support for TagMetadata #424
Conversation
6bbd58b
to
4eab6b3
Compare
Codecov Report
@@ Coverage Diff @@
## master #424 +/- ##
==========================================
+ Coverage 94.88% 94.94% +0.05%
==========================================
Files 140 141 +1
Lines 9293 9333 +40
Branches 671 672 +1
==========================================
+ Hits 8818 8861 +43
+ Misses 475 472 -3
Continue to review full report at Codecov.
|
@@ -25,3 +25,38 @@ export interface TagValue { | |||
/** The value of a tag. */ | |||
readonly value: string; | |||
} | |||
|
|||
/** TagValueWithMetadata holds a TagValue and a TagMetadata. */ | |||
export interface TagValueWithMetadata { |
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.
If possible please don't expose this in the public APIs. In Java it's used as an internal implementation class (not part of API).
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.
Done, now this interface
is limited to core package only.
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, pending fix Yang requested
fe3e5d8
to
a511512
Compare
58d629b
to
d2c41b0
Compare
d2c41b0
to
beec862
Compare
Specs: https://github.com/census-instrumentation/opencensus-specs/blob/master/tags/TagMap.md#tagmetadata
This PR contains support for recording
TagMetadata
, there will be another PR after #326 to enforce check on propagating tags at Receiver and Sender level (https://github.com/census-instrumentation/opencensus-specs/blob/master/tags/TagMap.md#processing-at-receiver-and-sender).Fixes #296