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

Incorrect namespace issue #2558

Open
cijothomas opened this issue Nov 2, 2021 · 0 comments
Open

Incorrect namespace issue #2558

cijothomas opened this issue Nov 2, 2021 · 0 comments
Labels
bug Something isn't working need-majorversion-bump Anything which requires major version change due to breaking change.

Comments

@cijothomas
Copy link
Member

The following classes are in the root OpenTelemetry ns, as opposed to being in .Trace or .Logs namespace. Given this was not identified before 1.0.0 stable, changing this now would be breaking change. This issue is opened for tracking fixing this when we do a major bump.

https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry/SimpleActivityExportProcessor.cs#L19
https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry/BatchActivityExportProcessor.cs#L19
https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry/SimpleLogRecordExportProcessor.cs#L19
https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry/BatchLogRecordExportProcessor.cs#L19

@cijothomas cijothomas added the bug Something isn't working label Nov 2, 2021
@reyang reyang mentioned this issue Feb 22, 2022
@cijothomas cijothomas added the need-majorversion-bump Anything which requires major version change due to breaking change. label Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working need-majorversion-bump Anything which requires major version change due to breaking change.
Projects
None yet
Development

No branches or pull requests

1 participant