-
Notifications
You must be signed in to change notification settings - Fork 421
Open
Labels
Description
We have a SpanEvent class that is used to create a span. We now have to support OTel span events which makes this confusing because we already have a class named SpanEvent. And we add OTel span events on our node agent spans. This makes it confusing because now a span event has a span event on it.
When we added support for OTel span events, we created a class called TimedEvents because that's how OTel defines span events (and SpanEvent name class was already taken).
We should refactor our SpanEvent class to be something like Span or similar and rename TimeEvents to be called SpanEvents which makes more sense.
This PR needs to be merged first before we start work on this: #3600
Metadata
Metadata
Assignees
Labels
Type
Projects
Status
Triage Needed