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

Fix repeated string serialization for JSON. #6888

Merged
merged 2 commits into from
Nov 25, 2024

Add test case

b9176bf
Select commit
Loading
Failed to load commit list.
Merged

Fix repeated string serialization for JSON. #6888

Add test case
b9176bf
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Nov 22, 2024 in 0s

0.00% of diff hit (target 80.00%)

View this Pull Request on Codecov

0.00% of diff hit (target 80.00%)

Annotations

Check warning on line 127 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/JsonSerializer.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/JsonSerializer.java#L127 

Added line #L127 was not covered by tests

Check warning on line 136 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/JsonSerializer.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/JsonSerializer.java#L136 

Added line #L136 was not covered by tests

Check warning on line 139 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/JsonSerializer.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/JsonSerializer.java#L138-L139 

Added lines #L138 - L139 were not covered by tests

Check warning on line 116 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/MarshalerUtil.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/MarshalerUtil.java#L116 

Added line #L116 was not covered by tests

Check warning on line 118 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/MarshalerUtil.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/MarshalerUtil.java#L118 

Added line #L118 was not covered by tests

Check warning on line 120 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/MarshalerUtil.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/MarshalerUtil.java#L120 

Added line #L120 was not covered by tests

Check warning on line 169 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/ProtoSerializer.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/ProtoSerializer.java#L169 

Added line #L169 was not covered by tests

Check warning on line 171 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/ProtoSerializer.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/ProtoSerializer.java#L171 

Added line #L171 was not covered by tests

Check warning on line 230 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/Serializer.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/Serializer.java#L230 

Added line #L230 was not covered by tests

Check warning on line 233 in exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/Serializer.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

exporters/common/src/main/java/io/opentelemetry/exporter/internal/marshal/Serializer.java#L232-L233 

Added lines #L232 - L233 were not covered by tests
View more details on Codecov