feat: add transform processor as opt-in on custom build #1901
Conversation
The committers listed above are authorized under a signed CLA. |
petrusdemelo
force-pushed
the
transformprocessor
branch
from
56f1fc3 to
5eda47e
Compare
|
@petrusdemelo Shouldn't this new processor be added to default list here: https://github.com/open-telemetry/opentelemetry-lambda/blob/main/collector/lambdacomponents/default.go#L64? |
|
@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
left a comment
tylerbenson
left a comment
There was a problem hiding this comment.
I don't have issue with this as long as @serkan-ozal is ok with it. Need to rebase and resolve the conflicts though.
|
@petrusdemelo Ok, makes sense. I am OK with the PR. Could you please resolve the conflicts, then we can merge it. |
|
@petrusdemelo any chance to rebase this PR? |
|
@serkan-ozal yes, I had busy weeks. Will do it this weekend. |
|
@petrusdemelo kindly bump on this |
3 participants
Description
Add
transform processoras opt-in for custom build usinglambdacomponents.processor.transformtag.