Output event name in SystemOutLogRecordExporter - #8609
Conversation
0791207 to
171d0bf
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8609 +/- ##
============================================
+ Coverage 91.62% 91.63% +0.01%
- Complexity 10321 10327 +6
============================================
Files 1013 1013
Lines 27287 27299 +12
Branches 3203 3206 +3
============================================
+ Hits 25001 25015 +14
+ Misses 1559 1558 -1
+ Partials 727 726 -1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
|
||
| ## Unreleased | ||
|
|
||
| ### SDK |
There was a problem hiding this comment.
No need to change this PR, but in future PRs no need to add a changelog entry. We generate changelog at release time.
There was a problem hiding this comment.
Thanks for the heads-up — I'll skip the changelog entry in future PRs.
|
Thank you for your contribution @thswlsqls! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. |
Pull request dashboard statusStatus: This pull request has been merged. |
Fixes #8608
Description
SystemOutLogRecordExporter.formatLog()did not print the log record's event name; it now appends[eventName: <name>]after the severity when one is set.LogRecord.EVENT_NAME(LogStatelessMarshaler,LogMarshaler).LogRecordData.getEventName()is a stable log data-model field (stabilized in Stabilize log record event name #7277); this exporter was not updated when the field was added.Testing done
SystemOutLogRecordExporterTest#formatWithEventNamefor the event-name-present case; the existingformattest covers the absent case../gradlew :exporters:logging:check— 16 tests passed.## UnreleasedCHANGELOG entry is included. No public API change (formatLogis package-private), so no apidiff update.Related: PR #6591 updated
formatLog's body field when the data model changed.