-
Notifications
You must be signed in to change notification settings - Fork 774
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 event API #4781
Add event API #4781
Conversation
Codecov ReportBase: 90.78% // Head: 90.76% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #4781 +/- ##
============================================
- Coverage 90.78% 90.76% -0.03%
- Complexity 4822 4835 +13
============================================
Files 554 554
Lines 14383 14414 +31
Branches 1397 1399 +2
============================================
+ Hits 13058 13083 +25
- Misses 909 911 +2
- Partials 416 420 +4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
api/logs/src/main/java/io/opentelemetry/api/logs/DefaultLogger.java
Outdated
Show resolved
Hide resolved
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.
I'm not 100% convinced that this API will end up being particularly user-friendly, but I'd like to get it into the project and try it out and see how it feels when used "in anger" before I pass final judgement on it.
We'll see how it goes! I'm not particularly fond of having to use a bunch of |
Resolves #4714.