-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add SpanID data type #1855
Add SpanID data type #1855
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1855 +/- ##
==========================================
- Coverage 91.27% 91.25% -0.02%
==========================================
Files 274 274
Lines 16392 16392
==========================================
- Hits 14961 14959 -2
- Misses 1001 1002 +1
- Partials 430 431 +1
Continue to review full report at Codecov.
|
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.
Can you please rebase this? I think the bytesid was merged.
This is step 2 in adding SpanID custom data type. This only introduces the type but does not yet use it in Protobuf messages. It will added in a future PR. Contributes to open-telemetry#1177
af57a27
to
a60f8a6
Compare
@bogdandrutu Rebased, PTAL. |
* remove Set.Encoded(Encoder) enconding cache Co-authored-by: Tyler Yahn <[email protected]>
This is step 2 in adding SpanID custom data type. This only introduces the
type but does not yet use it in Protobuf messages. It will added in a future PR.
Contributes to #1177