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

Jump to bottom

[Logs SDK] Send resource once to processor and exporter, and not for every event. #1636

Merged

cijothomas merged 34 commits into open-telemetry:main from lalitb:set-resource-optimize

Apr 30, 2024

Merged

[Logs SDK] Send resource once to processor and exporter, and not for every event. #1636

leftover commit

Codecov / codecov/patch failed Mar 29, 2024 in 1s

55.7% of diff hit (target 68.9%)

View this Pull Request on Codecov

55.7% of diff hit (target 68.9%)

Annotations

Check warning on line 22 in opentelemetry-otlp/src/exporter/http/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/http/logs.rs#L22

Added line #L22 was not covered by tests

Check warning on line 56 in opentelemetry-otlp/src/exporter/http/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/http/logs.rs#L54-L56

Added lines #L54 - L56 were not covered by tests

Check warning on line 63 in opentelemetry-otlp/src/exporter/http/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/http/logs.rs#L60-L63

Added lines #L60 - L63 were not covered by tests

Check warning on line 69 in opentelemetry-otlp/src/exporter/http/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/http/logs.rs#L66-L69

Added lines #L66 - L69 were not covered by tests

Check warning on line 71 in opentelemetry-otlp/src/exporter/http/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/http/logs.rs#L71

Added line #L71 was not covered by tests

Check warning on line 277 in opentelemetry-otlp/src/exporter/http/mod.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/http/mod.rs#L277

Added line #L277 was not covered by tests

Check warning on line 48 in opentelemetry-otlp/src/exporter/tonic/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/tonic/logs.rs#L48

Added line #L48 was not covered by tests

Check warning on line 75 in opentelemetry-otlp/src/exporter/tonic/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/tonic/logs.rs#L68-L75

Added lines #L68 - L75 were not covered by tests

Check warning on line 80 in opentelemetry-otlp/src/exporter/tonic/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/tonic/logs.rs#L80

Added line #L80 was not covered by tests

Check warning on line 94 in opentelemetry-otlp/src/exporter/tonic/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/exporter/tonic/logs.rs#L92-L94

Added lines #L92 - L94 were not covered by tests

Check warning on line 109 in opentelemetry-otlp/src/logs.rs

codecov / codecov/patch

opentelemetry-otlp/src/logs.rs#L107-L109

Added lines #L107 - L109 were not covered by tests

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

codecov / codecov/patch

opentelemetry-proto/src/transform/common.rs#L34-L39

Added lines #L34 - L39 were not covered by tests

Check warning on line 126 in opentelemetry-proto/src/transform/logs.rs

codecov / codecov/patch

opentelemetry-proto/src/transform/logs.rs#L119-L126

Added lines #L119 - L126 were not covered by tests

Check warning on line 129 in opentelemetry-proto/src/transform/logs.rs

codecov / codecov/patch

opentelemetry-proto/src/transform/logs.rs#L129

Added line #L129 was not covered by tests

Check warning on line 132 in opentelemetry-proto/src/transform/logs.rs

codecov / codecov/patch

opentelemetry-proto/src/transform/logs.rs#L132

Added line #L132 was not covered by tests

Check warning on line 25 in opentelemetry-sdk/src/export/logs/mod.rs

codecov / codecov/patch

opentelemetry-sdk/src/export/logs/mod.rs#L25

Added line #L25 was not covered by tests

Check warning on line 57 in opentelemetry-sdk/src/logs/log_processor.rs

codecov / codecov/patch

opentelemetry-sdk/src/logs/log_processor.rs#L57

Added line #L57 was not covered by tests

Check warning on line 513 in opentelemetry-sdk/src/logs/log_processor.rs

codecov / codecov/patch

opentelemetry-sdk/src/logs/log_processor.rs#L511-L513

Added lines #L511 - L513 were not covered by tests

Check warning on line 50 in opentelemetry-stdout/src/logs/exporter.rs

codecov / codecov/patch

opentelemetry-stdout/src/logs/exporter.rs#L49-L50

Added lines #L49 - L50 were not covered by tests

Check warning on line 63 in opentelemetry-stdout/src/logs/exporter.rs

codecov / codecov/patch

opentelemetry-stdout/src/logs/exporter.rs#L61-L63

Added lines #L61 - L63 were not covered by tests

Check warning on line 137 in opentelemetry-stdout/src/logs/exporter.rs

codecov / codecov/patch

opentelemetry-stdout/src/logs/exporter.rs#L137

Added line #L137 was not covered by tests

Check warning on line 29 in opentelemetry-stdout/src/logs/transform.rs

codecov / codecov/patch

opentelemetry-stdout/src/logs/transform.rs#L24-L29

Added lines #L24 - L29 were not covered by tests

Check warning on line 33 in opentelemetry-stdout/src/logs/transform.rs

codecov / codecov/patch

opentelemetry-stdout/src/logs/transform.rs#L33

Added line #L33 was not covered by tests

Check warning on line 36 in opentelemetry-stdout/src/logs/transform.rs

codecov / codecov/patch

opentelemetry-stdout/src/logs/transform.rs#L36

Added line #L36 was not covered by tests

Check warning on line 39 in opentelemetry-stdout/src/logs/transform.rs

codecov / codecov/patch

opentelemetry-stdout/src/logs/transform.rs#L39

Added line #L39 was not covered by tests

View more details on Codecov