Skip to content

sdk/metric: ignore nil ExemplarFilter in WithExemplarFilter#8279

Open
kushalShukla-web wants to merge 2 commits intoopen-telemetry:mainfrom
kushalShukla-web:feature/nil_filter
Open

sdk/metric: ignore nil ExemplarFilter in WithExemplarFilter#8279
kushalShukla-web wants to merge 2 commits intoopen-telemetry:mainfrom
kushalShukla-web:feature/nil_filter

Conversation

@kushalShukla-web
Copy link
Copy Markdown

@kushalShukla-web kushalShukla-web commented Apr 29, 2026

fixes : #8240

Hi @dashpole , could you please take a look when you have time?

@kushalShukla-web kushalShukla-web changed the title Signed-off-by: Kushal Shukla <kushalshukla110@gmail.com> sdk/metric: ignore nil ExemplarFilter in WithExemplarFilter Apr 29, 2026
@kushalShukla-web kushalShukla-web marked this pull request as draft April 29, 2026 18:48
@kushalShukla-web kushalShukla-web marked this pull request as ready for review May 1, 2026 15:59
@dashpole
Copy link
Copy Markdown
Contributor

dashpole commented May 1, 2026

I don't think we've made a decision on the issue, which lists two possible resolutions.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Decide how to handle metric.WithExemplarFilter(nil)

2 participants