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

Clarify view conflicts #2462

Merged
merged 14 commits into from
Apr 13, 2022
Merged

Clarify view conflicts #2462

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
2122ce5
Clarify view conflicts
jack-berg Mar 31, 2022
2e0176c
SDKs may fail fast on view registration conflict
jack-berg Mar 31, 2022
415e7df
Conflicting views produce metric identity conflicts
jack-berg Mar 31, 2022
97e43af
Update specification/metrics/sdk.md
jmacd Apr 1, 2022
2695c7d
Clarify view semantic error behavior
jack-berg Apr 6, 2022
13bd2fb
Merge branch 'main' of github.com:open-telemetry/opentelemetry-specif…
jack-berg Apr 6, 2022
0151177
Update changelog
jack-berg Apr 6, 2022
336172c
Merge branch 'view-conflict' of https://github.com/jack-berg/opentele…
jack-berg Apr 6, 2022
e4afe10
Merge branch 'main' of github.com:open-telemetry/opentelemetry-specif…
jack-berg Apr 8, 2022
67def03
Merge branch 'main' into view-conflict
reyang Apr 13, 2022
1194139
Merge branch 'main' into view-conflict
reyang Apr 13, 2022
ef2691b
Merge branch 'main' of github.com:open-telemetry/opentelemetry-specif…
jack-berg Apr 13, 2022
ba43060
Fix merge
jack-berg Apr 13, 2022
6307e44
Merge branch 'view-conflict' of https://github.com/jack-berg/opentele…
jack-berg Apr 13, 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
12 changes: 7 additions & 5 deletions specification/metrics/sdk.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -210,11 +210,13 @@ made with an Instrument:
* If the `MeterProvider` has one or more `View`(s) registered:
* For each View, if the Instrument could match the instrument selection
criteria:
* Try to apply the View configuration. If there is an error or a conflict
(e.g. the View requires to export the metrics using a certain name, but
the name is already used by another View), provide a way to let the user
know (e.g. expose
[self-diagnostics logs](../error-handling.md#self-diagnostics)).
* Try to apply the View configuration. If applying the View results
cijothomas marked this conversation as resolved.
Show resolved Hide resolved
in [duplicate instruments](./api.md#instrument-type-conflict-detection)
with the same name, the implementation SHOULD apply the View and emit a
warning. If it is not possible to apply the View without producing
reyang marked this conversation as resolved.
Show resolved Hide resolved
semantic errors (e.g. the View sets an asynchronous instrument to use
the [Histogram aggregation](#histogram-aggregation)) the implementation
SHOULD drop the View and emit a warning.
reyang marked this conversation as resolved.
Show resolved Hide resolved
jack-berg marked this conversation as resolved.
Show resolved Hide resolved
* If the Instrument could not match with any of the registered `View`(s), the
SDK SHOULD enable the instrument using the default aggregation and temporality.
Users can configure match-all Views using [Drop aggregation](#drop-aggregation)
Expand Down