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

Cache instruments so repeatedly creating identical instruments doesn't leak memory #4820

Merged

MrAlias merged 6 commits into open-telemetry:main from dashpole:duplicate_instruments

Jan 24, 2024

+284 −50

Merged

Cache instruments so repeatedly creating identical instruments doesn't leak memory #4820

Merge branch 'main' into duplicate_instruments

Codecov / codecov/patch succeeded Jan 24, 2024 in 0s

83.3% of diff hit (target 82.3%)

View this Pull Request on Codecov

83.3% of diff hit (target 82.3%)

Annotations

Check warning on line 133 in sdk/metric/meter.go

codecov / codecov/patch

sdk/metric/meter.go#L132-L133

Added lines #L132 - L133 were not covered by tests

Check warning on line 142 in sdk/metric/meter.go

codecov / codecov/patch

sdk/metric/meter.go#L141-L142

Added lines #L141 - L142 were not covered by tests

Check warning on line 266 in sdk/metric/meter.go

codecov / codecov/patch

sdk/metric/meter.go#L265-L266

Added lines #L265 - L266 were not covered by tests

Check warning on line 275 in sdk/metric/meter.go

codecov / codecov/patch

sdk/metric/meter.go#L274-L275

Added lines #L274 - L275 were not covered by tests

Check warning on line 380 in sdk/metric/meter.go

codecov / codecov/patch

sdk/metric/meter.go#L373-L380

Added lines #L373 - L380 were not covered by tests

View more details on Codecov