Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix all TraceExporter usages, use TracesExporter #2937

Merged
merged 2 commits into from
Apr 14, 2021

Conversation

bogdandrutu
Copy link
Member

Signed-off-by: Bogdan Drutu bogdandrutu@gmail.com

@bogdandrutu bogdandrutu requested a review from a team April 14, 2021 20:32
@codecov
Copy link

codecov bot commented Apr 14, 2021

Codecov Report

Merging #2937 (8077899) into main (6458914) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2937   +/-   ##
=======================================
  Coverage   91.67%   91.67%           
=======================================
  Files         312      312           
  Lines       15319    15319           
=======================================
  Hits        14043    14043           
  Misses        870      870           
  Partials      406      406           
Impacted Files Coverage Δ
exporter/exporterhelper/tracehelper.go 100.00% <ø> (ø)
exporter/opencensusexporter/opencensus.go 86.66% <ø> (ø)
service/internal/builder/exporters_builder.go 85.86% <ø> (ø)
exporter/exporterhelper/factory.go 100.00% <100.00%> (ø)
exporter/fileexporter/factory.go 100.00% <100.00%> (ø)
exporter/jaegerexporter/exporter.go 94.66% <100.00%> (ø)
exporter/jaegerexporter/factory.go 91.30% <100.00%> (ø)
exporter/kafkaexporter/factory.go 100.00% <100.00%> (ø)
exporter/loggingexporter/factory.go 73.68% <100.00%> (ø)
exporter/loggingexporter/logging_exporter.go 96.40% <100.00%> (ø)
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6458914...8077899. Read the comment docs.

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
@bogdandrutu bogdandrutu merged commit 0f75efa into open-telemetry:main Apr 14, 2021
@bogdandrutu bogdandrutu deleted the tracesexp branch April 14, 2021 21:36
hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this pull request Apr 27, 2023
…lemetry#2937)

Bumps [github.com/apache/pulsar-client-go](https://github.com/apache/pulsar-client-go) from 0.9.0 to 0.10.0.
- [Release notes](https://github.com/apache/pulsar-client-go/releases)
- [Changelog](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG.md)
- [Commits](apache/pulsar-client-go@v0.9.0...v0.10.0)

---
updated-dependencies:
- dependency-name: github.com/apache/pulsar-client-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants