Skip to content

Conversation

nosan
Copy link
Contributor

@nosan nosan commented Mar 25, 2025

See #44645

Also helps with that #36248 (comment)

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 25, 2025
@mhalbritter mhalbritter added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 26, 2025
@mhalbritter mhalbritter added this to the 3.5.x milestone Mar 26, 2025
@mhalbritter
Copy link
Contributor

Thanks @nosan. I think that's a good idea.

@nosan
Copy link
Contributor Author

nosan commented Mar 26, 2025

@mhalbritter
I think it would be nice to merge #44789 before this one. That way, you would avoid any merge conflicts when merging 3.4.x -> main later. Once #44789 gets merged, I will rebase this one. WDYT?

@mhalbritter
Copy link
Contributor

Sure. #44789 is merged.

This commit introduces customizers for both
OtlpHttpSpanExporterBuilder and OtlpGrpcSpanExporterBuilder.

Signed-off-by: Dmytro Nosan <[email protected]>
@nosan
Copy link
Contributor Author

nosan commented Mar 26, 2025

Thanks, @mhalbritter.
I've rebased this one.

@mhalbritter mhalbritter changed the title Introduce OtlpGrpcSpanExporterBuilderCustomizer and OtlpHttpSpanExporterBuilderCustomizer customizers Add OtlpGrpcSpanExporterBuilderCustomizer and OtlpHttpSpanExporterBuilderCustomizer customizers Mar 26, 2025
@mhalbritter mhalbritter self-assigned this Mar 26, 2025
mhalbritter pushed a commit that referenced this pull request Mar 26, 2025
This commit adds customizers for both OtlpHttpSpanExporterBuilder
and OtlpGrpcSpanExporterBuilder.

See gh-44900

Signed-off-by: Dmytro Nosan <[email protected]>
@mhalbritter
Copy link
Contributor

Thanks @nosan !

@mhalbritter mhalbritter modified the milestones: 3.5.x, 3.5.0-RC1 Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants