Skip to content

Conversation

@lopes-felipe
Copy link

Description

Include the groupbyattrs as another processor option.

image

@cla-checker-service
Copy link

cla-checker-service bot commented May 15, 2025

💚 CLA has been signed

Felipe Lopes added 2 commits May 15, 2025 17:05
@lopes-felipe
Copy link
Author

Hey @edmocosta! Would you mind taking a look at this whenever you can? Thanks! :)

@edmocosta
Copy link
Collaborator

Hi @lopes-felipe, thank you for your PR, I have a limited bandwidth at the moment, but I will take a look at this as soon as possible :)

@edmocosta edmocosta self-requested a review May 21, 2025 16:00
@edmocosta edmocosta added the enhancement New feature or request label May 21, 2025
@lopes-felipe
Copy link
Author

Hi @lopes-felipe, thank you for your PR, I have a limited bandwidth at the moment, but I will take a look at this as soon as possible :)

No worries @edmocosta, I just wanted you to be aware of this. Take the time you need :) (and great work, btw)

@lopes-felipe
Copy link
Author

@edmocosta did you have a chance to take a look at this?

@edmocosta
Copy link
Collaborator

Hi @lopes-felipe, I'm so sorry for the delay.
The PR looks good to me and it seems to be working fine, but there's still some internal discussions going on regarding supporting Collector components that are not OTTL-based, and what that would mean for the future of this project.
Please allow me a few more days, and I'll be back to you with a better answer. Thank you!

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants