-
Notifications
You must be signed in to change notification settings - Fork 956
[DO NOT MERGE] [Trace] Deprecate Record Exception API in favor of using Events #4825
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR deprecates the Record Exception API in the trace specification, aligning with the OTEP for Span Event API deprecation and the new semantic conventions that favor using Logs for recording exceptions.
- Marks the trace/exceptions.md document as deprecated with a reference to new semantic conventions
- Deprecates the
RecordExceptionAPI method in the trace API specification - Adds a CHANGELOG entry documenting the deprecation
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| specification/trace/exceptions.md | Changes status from Stable to Deprecated and directs users to the new semantic conventions for recording exceptions |
| specification/trace/api.md | Adds deprecation status to the Record Exception API section |
| CHANGELOG.md | Documents the deprecation of Record Exception API in the Traces section |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Cijo Thomas <[email protected]>
|
This PR was marked stale. It will be closed in 14 days without additional activity. |
Blocked by open-telemetry/semantic-conventions#3256
Blocked by #4824
Towards OTEP: Span Event API deprecation plan
Changes