Skip to content

Refactor classes for SpanEvent and TimedEvent #3603

@svetlanabrennan

Description

@svetlanabrennan

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

Labels

Type

No type

Projects

Status

In progress: Issues being worked on

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions