Doubt around Customising the OTLPMetricExporter

[atibdialpad]

Hi Team,
We are using the OTLPMetricExporter + PeriodicExportingMetricReader with a minor customisation. The CustomExporter is configured to "export" only when there are new data points as opposed to every 60s.

export default class CustomExporter extends OTLPMetricExporter {
  constructor (collectorOptions) {
    super(collectorOptions);
    this._newMetricsAvailable = false;
  }

  export (metrics, callback) {
    if (this._newMetricsAvailable) {
      this._newMetricsAvailable = false;
      return OTLPMetricExporter.prototype.export.call(this, metrics, callback);
    }
    ...
    ...
  }

  // This is called from add/record (i,e when we have actual data to report)
  setNewMetricsAvailable () {
    this._newMetricsAvailable = true;
  }
}

As a result what I see is cumulative data points timestamps which should look like this with a exportIntervalMillis of 60s (1m) :
(T0, T1m], (T0, T2m], (T0, T3m] now looks like (T0, T1m], (T0, T5m], (T0, T12m]

I wanted to understand if this is fine or does it violate any OpenTelemetry | Otel Collector principles. These metrics are being sent to an OtelCollector and them PRW to Grafana Mimir.

Any suggestions/feedback/advice is welcome. Thanks !!