search

open-telemetry / opentelemetry-python

OpenTelemetry Python API and SDK
https://opentelemetry.io
Apache License 2.0
1.8k stars 626 forks source link

Metric exporters should report and error condition... #2562

Open ocelotl opened 2 years ago

ocelotl commented 2 years ago

From the spec:

Metric Exporters SHOULD report an error condition for data output by the MetricReader with unsupported Aggregation or Aggregation Temporality, as this condition can be corrected by a change of MetricReader configuration.

aabmass commented 2 years ago

This requirement sounds like one for exporter authors, not the SDK. I don't think there's anything to do here

lzchen commented 2 years ago

@aabmass Does this apply to some of the exporters that we own? Let's say, Prometheus? Also how would exporter authors even know about this requirement during implementation? Should we take responsibility and abstract this behavior?