Skip to content

[intervalprocessor] Add delta temporality metrics support - #50922

Open
SumitDalavi wants to merge 1 commit into
open-telemetry:mainfrom
SumitDalavi:issue-50845-delta-temporality-intervalprocessor
Open

[intervalprocessor] Add delta temporality metrics support#50922
SumitDalavi wants to merge 1 commit into
open-telemetry:mainfrom
SumitDalavi:issue-50845-delta-temporality-intervalprocessor

Conversation

@SumitDalavi

Copy link
Copy Markdown
Contributor

Description

Currently, intervalprocessor ignores delta temporality metrics and simply passes them through. This PR adds support for accumulating delta metric values across the interval using a local copy of the deltatocumulativeprocessor's data addition utilities.

To maintain the "smallest maintainable change" principle and avoid breaking the existing deltatocumulativeprocessor, the data-addition logic was localized into intervalprocessor. (We can extract this into a shared internal/exp/metrics/data package in a future PR if desired).

Link to tracking issue

Fixes #50845

Testing

  • Updated existing intervalprocessor tests to assert that delta metrics are properly aggregated over intervals.
  • Verified all targeted component tests in processor/intervalprocessor/... pass cleanly.

Documentation

  • N/A

Authorship

  • I, a human, wrote this pull request description myself.

Currently, intervalprocessor ignores delta temporality metrics and simply passes them through. This PR adds support for accumulating delta metric values across the interval using a local copy of the deltatocumulativeprocessor's data addition utilities.

Fixes open-telemetry#50845.
@SumitDalavi
SumitDalavi force-pushed the issue-50845-delta-temporality-intervalprocessor branch from 3ba8c72 to 748f3a9 Compare September 11, 2026 16:28
@SumitDalavi

Copy link
Copy Markdown
Contributor Author

Hi! I've opened this PR to add delta temporality metrics support for the \intervalprocessor. To keep this as a minimal and safe change (without risking regressions in the existing \deltatocumulativeprocessor), I duplicated the data-addition logic into \intervalprocessor/internal/data.

However, as a future improvement or if reviewers prefer doing it in this PR, I propose we extract these data-addition utilities into a shared internal package (e.g., \internal/exp/metrics/data).

Looking forward to your feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: delta temporality metrics support in intervalprocessor

1 participant