-
Notifications
You must be signed in to change notification settings - Fork 67
Contribute OTLP UDP Exporter #1035
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
Conversation
...rter/src/main/java/software/opentelemetry/exporters/otlp/udp/OtlpUdpSpanExporterBuilder.java
Outdated
Show resolved
Hide resolved
…dy working in Lambda environment or not
…version in the build file
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1035 +/- ##
=============================================
- Coverage 85.71% 69.49% -16.22%
- Complexity 19 325 +306
=============================================
Files 3 26 +23
Lines 49 1177 +1128
Branches 5 166 +161
=============================================
+ Hits 42 818 +776
- Misses 3 304 +301
- Partials 4 55 +51 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
cc6f38c to
6fb094a
Compare
|
Apart from the open comments, the rest of the PR lgtm! |
…g a separate settings.gradle file for our exporter
.../java/software/amazon/opentelemetry/exporters/otlp/udp/trace/OtlpUdpSpanExporterBuilder.java
Outdated
Show resolved
Hide resolved
.../java/software/amazon/opentelemetry/exporters/otlp/udp/trace/OtlpUdpSpanExporterBuilder.java
Outdated
Show resolved
Hide resolved
.../java/software/amazon/opentelemetry/exporters/otlp/udp/trace/OtlpUdpSpanExporterBuilder.java
Show resolved
Hide resolved
…ivate cmethod and reuse it again wherever needed
43f06f8 to
41e00ff
Compare
41e00ff to
47b8655
Compare
be01061 to
d997980
Compare
jj22ee
left a comment
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.
fyi, I don't see the unit tests automatically running in the workflows anymore.
Given you have already run the unit tests successfully locally, and the future release workflow will run the unit tests, let's get this merged as to not delay this PR further. I'll followup with a workflow that runs the unit tests when the steps to publish this exporter are added.
Issue #, if available:
Description of changes:
Contributing our OTLP UDP exporter.
Testing:


Published the package to MavenLocal and tried running the sample app
Ran the unit tests as well
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.