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

Jump to bottom

Move specification for implementations of the metric API into own specification #3171

Merged
merged 24 commits into from
Feb 14, 2023
Merged

Move specification for implementations of the metric API into own specification #3171

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
dc018df
Add the initial draft of the metric No-Op
MrAlias Jan 24, 2023
bdd82fb
Give overview of SDK
MrAlias Jan 24, 2023
e2b62b6
Move API spec of MeterProvider creation to SDK
MrAlias Jan 24, 2023
ee4bfd7
Merge branch 'main' into noop-metric
MrAlias Jan 30, 2023
343eabe
Move SDK spec for meter out of API
MrAlias Feb 2, 2023
2e22500
Merge branch 'main' into noop-metric
MrAlias Feb 2, 2023
c356980
Use consistent emphasis style
MrAlias Feb 2, 2023
6c83c87
Fix toc addition from merge
MrAlias Feb 2, 2023
3213e5a
Generate toc for noop.md
MrAlias Feb 2, 2023
fb27024
Remove API req to treat empty unit/desc as ""
MrAlias Feb 2, 2023
640d3a6
Merge branch 'main' into noop-metric
MrAlias Feb 6, 2023
0a76c8b
Merge branch 'main' into noop-metric
MrAlias Feb 6, 2023
16f1656
Merge branch 'main' into noop-metric
MrAlias Feb 7, 2023
7a841d2
Merge branch 'main' into noop-metric
MrAlias Feb 9, 2023
71e52f2
Apply suggestions from code review
MrAlias Feb 9, 2023
a04838a
Update Meter Creation from code review
MrAlias Feb 9, 2023
aee8049
Link info on instrument identifying fields
MrAlias Feb 9, 2023
77d0d96
Merge branch 'main' into noop-metric
MrAlias Feb 9, 2023
ff3968f
Fix singular possessive
MrAlias Feb 9, 2023
e17384d
Apply suggestions from code review
MrAlias Feb 13, 2023
815f59f
Merge branch 'main' into noop-metric
MrAlias Feb 13, 2023
cb73f45
Require all language impls to provide a No-Op
MrAlias Feb 13, 2023
befae32
Remove SDK mention from unit/desc treatment
MrAlias Feb 14, 2023
ffa3e3f
Merge branch 'main' into noop-metric
MrAlias Feb 14, 2023
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
Remove SDK mention from unit/desc treatment
  • Loading branch information
@MrAlias
MrAlias committed Feb 14, 2023
commit befae3264d7ca4a1dcea0e0acd92c598d3699fb9
6 changes: 3 additions & 3 deletions specification/metrics/api.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -212,8 +212,8 @@ DIGIT = %x30-39 ; 0-9

#### Instrument unit

The `unit` is an optional string provided by the author of the Instrument. It
SHOULD be treated as an opaque string from the API and SDK.
The `unit` is an optional string provided by the author of the Instrument. The
API SHOULD treat it as an opaque string.

* It MUST be case-sensitive (e.g. `kb` and `kB` are different units), ASCII
string.
Expand All @@ -225,7 +225,7 @@ SHOULD be treated as an opaque string from the API and SDK.
#### Instrument description

The `description` is an optional free-form text provided by the author of the
instrument. It MUST be treated as an opaque string from the API and SDK.
instrument. The API MUST treat it as an opaque string.

* It MUST support [BMP (Unicode Plane
0)](https://en.wikipedia.org/wiki/Plane_(Unicode)#Basic_Multilingual_Plane),
Expand Down