Skip to content

Commit dadad68

Browse files
author
Andrew Xue
committed
update events and links
1 parent 1834dc3 commit dadad68

File tree

1 file changed

+3
-9
lines changed
  • opentelemetry-sdk/src/opentelemetry/sdk/trace

1 file changed

+3
-9
lines changed

opentelemetry-sdk/src/opentelemetry/sdk/trace/__init__.py

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -252,10 +252,6 @@ class Span(trace_api.Span):
252252
this `Span`.
253253
"""
254254

255-
# Initialize these lazily assuming most spans won't have them.
256-
_empty_events = BoundedList(MAX_NUM_EVENTS)
257-
_empty_links = BoundedList(MAX_NUM_LINKS)
258-
259255
def __init__(
260256
self,
261257
name: str,
@@ -295,15 +291,15 @@ def __init__(
295291
)
296292

297293
if events is None:
298-
self.events = Span._empty_events
294+
self.events = BoundedList(MAX_NUM_EVENTS)
299295
else:
300296
self.events = BoundedList(MAX_NUM_EVENTS)
301297
for event in events:
302298
self._filter_attribute_values(event.attributes)
303299
self.events.append(event)
304300

305301
if links is None:
306-
self.links = Span._empty_links
302+
self.links = BoundedList(MAX_NUM_LINKS)
307303
else:
308304
self.links = BoundedList.from_seq(MAX_NUM_LINKS, links)
309305

@@ -438,9 +434,7 @@ def _add_event(self, event: EventBase) -> None:
438434
if not self.is_recording_events():
439435
return
440436
has_ended = self.end_time is not None
441-
if not has_ended:
442-
if self.events is Span._empty_events:
443-
self.events = BoundedList(MAX_NUM_EVENTS)
437+
444438
if has_ended:
445439
logger.warning("Calling add_event() on an ended span.")
446440
return

0 commit comments

Comments
 (0)