Skip to content

Allow duplicate instrument registration #2916

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

Merged
cijothomas merged 30 commits into from
Mar 5, 2022
Merged

Allow duplicate instrument registration #2916

Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
ac0e766
Allow duplicate instrument registration
alanwest Feb 18, 2022
7fb51d0
Merge branch 'main' into alanwest/metric-identity
alanwest Feb 18, 2022
27b595a
Merge branch 'main' into alanwest/metric-identity
cijothomas Feb 23, 2022
ceacd5f
Merge branch 'main' into alanwest/metric-identity
alanwest Feb 23, 2022
d4a45d0
Rename MetricIdentity to InstrumentIdentity
alanwest Feb 23, 2022
0423b57
Handle duplicate instrument registration on views code path
alanwest Feb 25, 2022
56e5313
Test identical instruments with no views
alanwest Feb 25, 2022
024e947
Merge branch 'main' into alanwest/metric-identity
alanwest Feb 25, 2022
134fc1b
Merge branch 'alanwest/metric-identity' of github.com:alanwest/opente…
alanwest Feb 25, 2022
dd19e28
Allow distinct metric streams with same name
alanwest Mar 1, 2022
d399bee
Add more tests
alanwest Mar 1, 2022
445a3da
Merge branch 'main' into alanwest/metric-identity
alanwest Mar 1, 2022
8062437
Update warning message
alanwest Mar 1, 2022
37526c0
Merge branch 'main' into alanwest/metric-identity
alanwest Mar 1, 2022
16b9b63
Fixx spelling
alanwest Mar 1, 2022
1bfafc6
Update src/OpenTelemetry/Metrics/MetricReaderExt.cs
cijothomas Mar 2, 2022
3642a3a
Update changelog
alanwest Mar 3, 2022
c69ed26
Merge remote-tracking branch 'upstream/main' into alanwest/metric-ide…
alanwest Mar 3, 2022
8eec320
Hashtags: For more than just social media
alanwest Mar 3, 2022
2dd0058
Fix view use case
alanwest Mar 3, 2022
03a245e
Handle instruments with same name from meters with same name differen…
alanwest Mar 4, 2022
0ff1d22
Clear InstrumentIdenity map when meter is disposed
alanwest Mar 4, 2022
63bfe85
Merge remote-tracking branch 'upstream/main' into alanwest/metric-ide…
alanwest Mar 4, 2022
e52051d
Update public API
alanwest Mar 4, 2022
90ced8c
Update changelog
alanwest Mar 4, 2022
28aa3b3
Add additional clarification for duplicate instrument warning
alanwest Mar 4, 2022
247b0d7
Private set
alanwest Mar 4, 2022
683182e
Instrument name from InstrumentIdentity
alanwest Mar 4, 2022
ef836a7
Compute hash in constructor
alanwest Mar 5, 2022
012f080
Merge branch 'main' into alanwest/metric-identity
utpilla Mar 5, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update changelog
  • Loading branch information
@alanwest
alanwest committed Mar 4, 2022
commit 90ced8c5db49c4c44071b9b32180d64db229714d
4 changes: 4 additions & 0 deletions src/OpenTelemetry/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@

([#2916](https://github.com/open-telemetry/opentelemetry-dotnet/pull/2916))

* The `Meter` property on `OpenTelemetry.Metrics.Metric` has been removed.
It now has `MeterName` and `MeterVersion` properties.
([#2916](https://github.com/open-telemetry/opentelemetry-dotnet/pull/2916))

* Added support for implementing custom `ResourceDetector`.
([#2949](https://github.com/open-telemetry/opentelemetry-dotnet/pull/2949/)
[#2897](https://github.com/open-telemetry/opentelemetry-dotnet/pull/2897))
Expand Down