Skip to content

Add profiles support to kafka exporter#41369

Merged
songy23 merged 7 commits into
open-telemetry:mainfrom
dmathieu:profiles-kafka-exporter
Jul 22, 2025
Merged

Add profiles support to kafka exporter#41369
songy23 merged 7 commits into
open-telemetry:mainfrom
dmathieu:profiles-kafka-exporter

Conversation

@dmathieu
Copy link
Copy Markdown
Member

Description

This adds profiles support to the Kafka exporter (receiver coming in another PR).

}

func (e *kafkaProfilesMessenger) partitionData(ld pprofile.Profiles) iter.Seq2[[]byte, pprofile.Profiles] {
return func(yield func([]byte, pprofile.Profiles) bool) {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I didn't add partitioning by resource yet, as that's a bit trickier for profiles, and their dictionary. We should look into that in another PR.

@dmathieu dmathieu marked this pull request as ready for review July 17, 2025 14:25
@dmathieu dmathieu requested review from a team and MovieStoreGuy as code owners July 17, 2025 14:25
@github-actions github-actions Bot requested review from axw and pavolloffay July 17, 2025 14:25
Copy link
Copy Markdown
Contributor

@axw axw left a comment

Choose a reason for hiding this comment

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

Looks great, thank you!

@atoulme atoulme added the ready to merge Code review completed; ready to merge by maintainers label Jul 22, 2025
@songy23 songy23 merged commit 411d8fd into open-telemetry:main Jul 22, 2025
187 checks passed
@dmathieu dmathieu deleted the profiles-kafka-exporter branch July 23, 2025 07:28
povilasv pushed a commit to coralogix/opentelemetry-collector-contrib-old-fork that referenced this pull request Jul 29, 2025
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

This adds profiles support to the Kafka exporter (receiver coming in
another PR).
Dylan-M pushed a commit to Dylan-M/opentelemetry-collector-contrib that referenced this pull request Aug 5, 2025
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

This adds profiles support to the Kafka exporter (receiver coming in
another PR).
edmocosta pushed a commit that referenced this pull request Aug 27, 2025
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

This adds profiles support to the kafka receiver.

Related: the exporter PR:
#41369
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exporter/kafka ready to merge Code review completed; ready to merge by maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants