Skip to content
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.

Sign up for GitHub

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

Update KeyValue to avoid copying non-static string slices #2089

Closed
wants to merge 1 commit into from

Update KeyValue

91f0a1d
Select commit
Loading
Failed to load commit list.
Closed

Update KeyValue to avoid copying non-static string slices #2089

Update KeyValue
91f0a1d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Sep 10, 2024 in 0s

64.2% of diff hit (target 77.9%)

View this Pull Request on Codecov

64.2% of diff hit (target 77.9%)

Annotations

Check warning on line 148 in opentelemetry-proto/src/transform/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-proto/src/transform/common.rs#L146-L148 

Added lines #L146 - L148 were not covered by tests

Check warning on line 154 in opentelemetry-proto/src/transform/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-proto/src/transform/common.rs#L151-L154 

Added lines #L151 - L154 were not covered by tests

Check warning on line 163 in opentelemetry-proto/src/transform/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-proto/src/transform/common.rs#L163 

Added line #L163 was not covered by tests

Check warning on line 77 in opentelemetry-proto/src/transform/metrics.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-proto/src/transform/metrics.rs#L77 

Added line #L77 was not covered by tests

Check warning on line 86 in opentelemetry-proto/src/transform/metrics.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-proto/src/transform/metrics.rs#L86 

Added line #L86 was not covered by tests

Check warning on line 254 in opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampler.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampler.rs#L254 

Added line #L254 was not covered by tests

Check warning on line 132 in opentelemetry-stdout/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-stdout/src/common.rs#L130-L132 

Added lines #L130 - L132 were not covered by tests

Check warning on line 136 in opentelemetry-stdout/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-stdout/src/common.rs#L136 

Added line #L136 was not covered by tests

Check warning on line 139 in opentelemetry-stdout/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-stdout/src/common.rs#L139 

Added line #L139 was not covered by tests

Check warning on line 142 in opentelemetry-stdout/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-stdout/src/common.rs#L142 

Added line #L142 was not covered by tests

Check warning on line 145 in opentelemetry-stdout/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-stdout/src/common.rs#L145 

Added line #L145 was not covered by tests

Check warning on line 105 in opentelemetry-zipkin/src/exporter/mod.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-zipkin/src/exporter/mod.rs#L105 

Added line #L105 was not covered by tests

Check warning on line 97 in opentelemetry/src/baggage.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/baggage.rs#L94-L97 

Added lines #L94 - L97 were not covered by tests

Check warning on line 161 in opentelemetry/src/baggage.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/baggage.rs#L161 

Added line #L161 was not covered by tests

Check warning on line 264 in opentelemetry/src/baggage.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/baggage.rs#L262-L264 

Added lines #L262 - L264 were not covered by tests

Check warning on line 170 in opentelemetry/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/common.rs#L165-L170 

Added lines #L165 - L170 were not covered by tests

Check warning on line 172 in opentelemetry/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/common.rs#L172 

Added line #L172 was not covered by tests

Check warning on line 243 in opentelemetry/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/common.rs#L243 

Added line #L243 was not covered by tests

Check warning on line 245 in opentelemetry/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/common.rs#L245 

Added line #L245 was not covered by tests

Check warning on line 247 in opentelemetry/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/common.rs#L247 

Added line #L247 was not covered by tests

Check warning on line 498 in opentelemetry/src/common.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/common.rs#L498 

Added line #L498 was not covered by tests

Check warning on line 145 in opentelemetry/src/global/metrics.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/global/metrics.rs#L144-L145 

Added lines #L144 - L145 were not covered by tests

Check warning on line 229 in opentelemetry/src/global/trace.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/global/trace.rs#L229 

Added line #L229 was not covered by tests

Check warning on line 47 in opentelemetry/src/logs/logger.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/logs/logger.rs#L47 

Added line #L47 was not covered by tests

Check warning on line 31 in opentelemetry/src/logs/noop.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry/src/logs/noop.rs#L31 

Added line #L31 was not covered by tests
View more details on Codecov