-
Notifications
You must be signed in to change notification settings - Fork 825
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
feat(exporter-logs-otlp-proto): implements protobuf exporter for logs #3779
feat(exporter-logs-otlp-proto): implements protobuf exporter for logs #3779
Conversation
256b03d
to
e4a48b9
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3779 +/- ##
==========================================
+ Coverage 92.96% 92.98% +0.01%
==========================================
Files 299 300 +1
Lines 9082 9104 +22
Branches 1855 1862 +7
==========================================
+ Hits 8443 8465 +22
Misses 639 639
|
adbdf8f
to
1f8180e
Compare
...ental/packages/otlp-proto-exporter-base/src/platform/browser/OTLPProtoExporterBrowserBase.ts
Outdated
Show resolved
Hide resolved
3911bd0
to
2706824
Compare
...ental/packages/otlp-proto-exporter-base/src/platform/browser/OTLPProtoExporterBrowserBase.ts
Outdated
Show resolved
Hide resolved
experimental/packages/exporter-logs-otlp-proto/test/node/OTLPLogsExporter.test.ts
Outdated
Show resolved
Hide resolved
experimental/packages/exporter-logs-otlp-proto/test/browser/CollectorLogsExporter.test.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: Marc Pichler <[email protected]>
Co-authored-by: Marc Pichler <[email protected]>
Co-authored-by: Marc Pichler <[email protected]>
Co-authored-by: Marc Pichler <[email protected]>
…ogsExporter.test.ts Co-authored-by: Marc Pichler <[email protected]>
952f263
to
13e77df
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.
A few more comments, then this should be good to go 🙂
experimental/packages/exporter-logs-otlp-proto/src/platform/node/OTLPLogsExporter.ts
Outdated
Show resolved
Hide resolved
…de/OTLPLogsExporter.ts Co-authored-by: Marc Pichler <[email protected]>
Co-authored-by: Marc Pichler <[email protected]>
Co-authored-by: Marc Pichler <[email protected]>
Co-authored-by: Marc Pichler <[email protected]>
Which problem is this PR solving?
Adds a protobuf exporter for logs.
Short description of the changes
Create a new package for exporter-logs-otlp-proto
Type of change
How Has This Been Tested?
Tested in a browser app that uses the OTLPLogs exporter to export sample events
Checklist: