Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[processor/tailsamplingprocessor] move it to a stable component #10832

Closed
alexfdz opened this issue Jun 8, 2022 · 4 comments
Closed

[processor/tailsamplingprocessor] move it to a stable component #10832

alexfdz opened this issue Jun 8, 2022 · 4 comments
Labels
processor/tailsampling Tail sampling processor

Comments

@alexfdz
Copy link

alexfdz commented Jun 8, 2022

Is your feature request related to a problem? Please describe.
tailsamplingprocessor has been in beta for a while, is there any issue or missing feature we can help with?

It probably won't be included in distributions like aws-otel-collector if is not considered stable.

Describe the solution you'd like
tailsamplingprocessor to be considered a stable component.

Describe alternatives you've considered

Additional context
Probably related to #1797

@mx-psi mx-psi added the processor/tailsampling Tail sampling processor label Jun 8, 2022
@djaglowski
Copy link
Member

cc: @jpkrohling as code owner

@jpkrohling
Copy link
Member

I'm aware of a potential minimal leak I found while putting this under extreme pressure. Unfortunately, I wasn't able to find the cause of this potential leak. Despite that, I think we can indeed move it to stable, as I'm aware of people using it in production and haven't experienced this potential problem.

@jpkrohling
Copy link
Member

We are currently having discussions on what it means to be "stable". Given that "stable" might mean strict rules on the configuration properties, we might not want to move this to stable.

@jpkrohling
Copy link
Member

I'm closing this as I don't think we are ready to declare this stable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
processor/tailsampling Tail sampling processor
Projects
None yet
Development

No branches or pull requests

4 participants