-
Notifications
You must be signed in to change notification settings - Fork 812
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
Remove Status object from API, keep StatusCanonicalCode #1741
Conversation
sdk/tracing/src/main/java/io/opentelemetry/sdk/trace/data/ImmutableStatus.java
Outdated
Show resolved
Hide resolved
sdk/tracing/src/main/java/io/opentelemetry/sdk/trace/data/ImmutableStatus.java
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #1741 +/- ##
============================================
- Coverage 85.27% 85.17% -0.10%
+ Complexity 1363 1351 -12
============================================
Files 164 166 +2
Lines 5298 5276 -22
Branches 551 549 -2
============================================
- Hits 4518 4494 -24
+ Misses 579 578 -1
- Partials 201 204 +3
Continue to review full report at Codecov.
|
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
59a8462
to
67d7e1a
Compare
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.
Thanks!
Signed-off-by: Bogdan Drutu bogdandrutu@gmail.com