open-telemetry/opentelemetry-specification
Consistency of the term used to express data point kind, instrument kind
开放
#3,555 创建于 2023年6月14日
Stalearea:data-modelarea:sdkeditorialhelp wantedspec:metricstriage:accepted:ready
仓库指标
- 星标
- (4,267 个星标)
- PR 合并指标
- (PR 指标待抓取)
描述
What are you trying to achieve?
When the specification and the SDK documentation mention gauge, histogram, counter, and exponential histogram, it uses different terms to refer to the same concept. After searching the spec, I see the following terms have been used:
- Value type
- Data point type
- Data point kind
- Basic point kind
- Instrument kind
- Instrument type
It would be good to use the same term consistently throughout the specification (data-model.md) and the SDK documentation (sdk.md).
Additional context.
| Quotes | URL |
|---|---|
| ... Value type of the point (integer, floating point, etc) - here "value type" refers to whether the value is integer or float, and in the next sentence, "value type" refers to gauge, counter, histogram. | data-model.md#L248-L254 |
| The primary data of each timeseries are ordered (timestamp, value) points, with one of the following value types: Counter (Monotonic, Cumulative), Gauge, Histogram, Exponential Histogram | data-model.md#L248-L254 |
| The data point type (e.g. Sum, Gauge, Histogram ExponentialHistogram, Summary) | data-model.md#L283 |
Within certain data point types (e.g., Sum and Gauge) there is variation permitted in the numeric point value |
data-model.md#L302-L303 |
| The basic point kinds are: Sum, Gauge, Histogram, Exponential Histogram | data-model.md#L351-L363 |
| OTLP does not map 1:1 from its point types into timeseries points | data-model.md#L366 |
| The Default Aggregation informs the SDK to use the Instrument kind to select an aggregation | sdk.md#L415-L422 |
| The monotonicity of the aggregation is determined by the instrument type | sdk.md#L433-L440 |
| If the potential conflict involves instruments that can be distinguished by a supported View selector (e.g., instrument type) | sdk.md#L685-L686 |