Skip to content

Commit

Permalink
add sentence to highlight cumulative -> delta is also needed
Browse files Browse the repository at this point in the history
  • Loading branch information
huyan0 committed Aug 12, 2020
1 parent 22cf81c commit f1dbbd8
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 2 deletions.
Binary file removed .DS_Store
Binary file not shown.
Binary file removed text/.DS_Store
Binary file not shown.
Binary file removed text/metrics/.DS_Store
Binary file not shown.
4 changes: 2 additions & 2 deletions text/metrics/0131-otlp-export-behavior.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,9 @@ A discussed solution is to convert deltas to cumulative in the Collector both as

## Open questions

As stated in the previous section, delta to cumulative conversion in the Collector is needed to support Prometheus type backends. This may be necessary in the Collector in the future because the Collector may also accept metrics from other sources that report delta values.
As stated in the previous section, delta to cumulative conversion in the Collector is needed to support Prometheus type backends. This may be necessary in the Collector in the future because the Collector may also accept metrics from other sources that report delta values. On the other hand, if sources are reporting cumulative values, cumulative to delta conversion is needed to support Statsd type backends.

The future implementation for this conversion in the Collector is still under discussion. There is a proposal is to add a [Metric Aggregation Processor](https://github.com/open-telemetry/opentelemetry-collector/issues/1422) in the Collector which recommends a solution.
The future implementation for conversions in the Collector is still under discussion. There is a proposal is to add a [Metric Aggregation Processor](https://github.com/open-telemetry/opentelemetry-collector/issues/1422) in the Collector which recommends a solution for delta to cumulative conversion.

## Future possibilities

Expand Down

0 comments on commit f1dbbd8

Please sign in to comment.