Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sum aggregation function should support more types #37110

Closed
ernest-liftoff opened this issue Jan 9, 2025 · 2 comments
Closed

Sum aggregation function should support more types #37110

ernest-liftoff opened this issue Jan 9, 2025 · 2 comments
Labels
needs triage New item requiring triage processor/metricstransform Metrics Transform processor processor/transform Transform processor

Comments

@ernest-liftoff
Copy link

Component(s)

processor/metricstransform, processor/transform

Describe the issue you're reporting

NOTE: Only the sum aggregation function is supported for histogram and exponential histogram datatypes

This a new additional in processor/metricstransform, as it did not use to have this restriction. I understand that sum aggregation on sum types are incorrect, but that is only in the cumulative format. There should be no problem if the data is in the delta format.

Furthermore,  summing gauge (while it may or may not be correct), it should be allowed & up to user's discretion.

@ernest-liftoff ernest-liftoff added the needs triage New item requiring triage label Jan 9, 2025
@github-actions github-actions bot added processor/metricstransform Metrics Transform processor processor/transform Transform processor labels Jan 9, 2025
Copy link
Contributor

github-actions bot commented Jan 9, 2025

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@ernest-liftoff
Copy link
Author

sorry, misunderstood the statement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage New item requiring triage processor/metricstransform Metrics Transform processor processor/transform Transform processor
Projects
None yet
Development

No branches or pull requests

1 participant