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

[Trace API] Cleanup: Remove TraceError, TraceResult, and ExportError #2621

Open
lalitb opened this issue Feb 5, 2025 · 0 comments
Open

[Trace API] Cleanup: Remove TraceError, TraceResult, and ExportError #2621

lalitb opened this issue Feb 5, 2025 · 0 comments
Assignees
Labels
A-trace Area: issues related to tracing
Milestone

Comments

@lalitb
Copy link
Member

lalitb commented Feb 5, 2025

TraceError and TraceResult are defined in the OpenTelemetry API crate (source) and are used internally within the API implementation. Since the API itself should not return error objects, it would be preferable to either remove them from the API or move them to the SDK if they are needed there. The same consideration applies to ExportError.

@lalitb lalitb changed the title [Trace API] Error cleanup - TraceError, TraceResult, ExportError [Trace API] API should not expose errors: Consider removing TraceError, TraceResult, and ExportError Feb 5, 2025
@lalitb lalitb changed the title [Trace API] API should not expose errors: Consider removing TraceError, TraceResult, and ExportError [Trace API] API should not expose errors: consider removing TraceError, TraceResult, and ExportError Feb 5, 2025
@lalitb lalitb changed the title [Trace API] API should not expose errors: consider removing TraceError, TraceResult, and ExportError [Trace API] Cleanup: Remove TraceError, TraceResult, and ExportError Feb 5, 2025
@lalitb lalitb added the A-trace Area: issues related to tracing label Feb 5, 2025
@cijothomas cijothomas modified the milestones: Tracing API Stable, 0.29 Feb 6, 2025
@TommyCpp TommyCpp self-assigned this Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-trace Area: issues related to tracing
Projects
None yet
Development

No branches or pull requests

3 participants