Skip to content

Conversation

odubajDT
Copy link
Contributor

@odubajDT odubajDT commented Jan 15, 2025

Description

Introduce convert_summary_quantile_val_to_gauge() function which converts all quantiles of a Summary to Gauge with multiple datapoints, where each datapoint has an attribute representing the quantile.

Link to tracking issue

Fixes #33850

@github-actions github-actions bot added the processor/transform Transform processor label Jan 15, 2025
@odubajDT odubajDT force-pushed the transform-convert-summary-gauge branch 2 times, most recently from 96cccac to 81aa995 Compare January 16, 2025 11:43
@odubajDT odubajDT marked this pull request as ready for review January 16, 2025 12:07
@odubajDT odubajDT requested a review from a team as a code owner January 16, 2025 12:07
@odubajDT odubajDT force-pushed the transform-convert-summary-gauge branch from 81aa995 to b86a924 Compare January 17, 2025 09:52
Copy link
Contributor

github-actions bot commented Feb 8, 2025

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Feb 8, 2025
@odubajDT
Copy link
Contributor Author

odubajDT commented Feb 9, 2025

not stale

@odubajDT odubajDT requested a review from evan-bradley March 5, 2025 13:54
@atoulme
Copy link
Contributor

atoulme commented Mar 17, 2025

Looks like the next step is to have @evan-bradley review.

Copy link
Contributor

github-actions bot commented Jul 5, 2025

This PR was marked stale due to lack of activity. It will be closed in 14 days.

Copy link
Contributor

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot removed the Stale label Aug 1, 2025
@odubajDT odubajDT requested a review from evan-bradley August 5, 2025 06:30
Copy link
Contributor

@evan-bradley evan-bradley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks for your patience with the long review cycle @odubajDT.

@edmocosta @TylerHelmuth could one of you take a look?

…mary_quantile_val_to_gauge.go

Co-authored-by: Evan Bradley <[email protected]>
@evan-bradley evan-bradley merged commit 8f61805 into open-telemetry:main Aug 8, 2025
185 checks passed
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.

[processor/transform] Allow specifying metric name suffix when using convert summary functions

5 participants