Skip to content

feat: add transform processor as opt-in on custom build#1901

Closed
petrusdemelo wants to merge 1 commit into
open-telemetry:mainfrom
petrusdemelo:transformprocessor
Closed

feat: add transform processor as opt-in on custom build#1901
petrusdemelo wants to merge 1 commit into
open-telemetry:mainfrom
petrusdemelo:transformprocessor

Conversation

@petrusdemelo

Copy link
Copy Markdown

Description

Add transform processor as opt-in for custom build using lambdacomponents.processor.transform tag.

@petrusdemelo petrusdemelo requested a review from a team as a code owner July 14, 2025 17:30
@linux-foundation-easycla

linux-foundation-easycla Bot commented Jul 14, 2025

Copy link
Copy Markdown

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: petrusdemelo / name: Petrus de Melo (625c229)

@serkan-ozal

Copy link
Copy Markdown
Contributor

@petrusdemelo

Copy link
Copy Markdown
Author

@serkan-ozal I did not want to change the default build for the lambda collector. I just wanted to have it as an opt in option for custom build. But I can definitely add in the default build if that is the desire.

@tylerbenson tylerbenson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't have issue with this as long as @serkan-ozal is ok with it. Need to rebase and resolve the conflicts though.

@serkan-ozal

Copy link
Copy Markdown
Contributor

@petrusdemelo Ok, makes sense. I am OK with the PR. Could you please resolve the conflicts, then we can merge it.

@tylerbenson tylerbenson added the go Pull requests that update Go code label Oct 15, 2025
@serkan-ozal

Copy link
Copy Markdown
Contributor

@petrusdemelo any chance to rebase this PR?

@petrusdemelo

Copy link
Copy Markdown
Author

@serkan-ozal yes, I had busy weeks. Will do it this weekend.

@serkan-ozal

Copy link
Copy Markdown
Contributor

@petrusdemelo kindly bump on this

@ncorreia

ncorreia commented Jan 6, 2026

Copy link
Copy Markdown

Hello, it would be really useful to be able to use a transform processor. Any updates on this? Thanks!

@petrusdemelo

Copy link
Copy Markdown
Author

updated!

@wpessers

wpessers commented Mar 3, 2026

Copy link
Copy Markdown
Member

@petrusdemelo I haven’t looked at this pr since review was done by other maintainers. Any reason for closing the pr? I see @RaphaelManke has shown interest in this feature in his comment. So if there’s still value in this I’m willing to reopen the PR and take a quick look myself and merge it if everything looks good.

@petrusdemelo

Copy link
Copy Markdown
Author

@wpessers This branch has gone through several merge conflicts already, and since the PR has been open for quite a while, I haven’t been able to keep up with resolving each new conflict as they come up. If you’ll have time to take a look and merge it, I’d be happy to reopen the PR and fix the current conflicts.

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

Labels

go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants