Skip to content

Conversation

@IrisTuntun
Copy link
Contributor

  1. Fixed the typo testCollectorEnpoint to testCollectorEndpoint.
  2. Added assertions on process tags in gen.Batch that was exported in TestNewExporterPipelineWithOptions.

Fixes #1699
Note: gen.Batch has already been recorded in testCollectorEndpoint thanks to #1693

@codecov
Copy link

codecov bot commented Mar 30, 2021

Codecov Report

Merging #1753 (76e5d2e) into main (ecc635d) will increase coverage by 0.0%.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #1753   +/-   ##
=====================================
  Coverage   78.6%   78.6%           
=====================================
  Files        133     133           
  Lines       7072    7072           
=====================================
+ Hits        5560    5564    +4     
+ Misses      1267    1265    -2     
+ Partials     245     243    -2     
Impacted Files Coverage Δ
sdk/metric/sdk.go 81.3% <0.0%> (+1.1%) ⬆️
sdk/metric/refcount_mapped.go 100.0% <0.0%> (+22.2%) ⬆️

@Aneurysm9 Aneurysm9 added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Mar 30, 2021
@Aneurysm9 Aneurysm9 merged commit 3947cab into open-telemetry:main Apr 1, 2021
@pellared pellared added this to the untracked milestone Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test exporting a batch of spans from Jaeger Exporter is working as expected

5 participants