opentelemetry-sdk: update iterator for BoundedList

.changelog/5297.changed

@@ -0,0 +1 @@
+opentelemetry-sdk: update iterator for BoundedList

opentelemetry-api/src/opentelemetry/attributes/__init__.py

@@ -307,9 +307,9 @@ def __delitem__(self, key: str) -> None:
         with self._lock:
             del self._dict[key]
 
-    def __iter__(self):  # type: ignore
+    def __iter__(self):
         with self._lock:
-            return iter(list(self._dict))  # type: ignore
+            return iter(list(self._dict))
 
     def __len__(self) -> int:
         return len(self._dict)

opentelemetry-sdk/benchmarks/trace/test_benchmark_trace.py

@@ -172,6 +172,18 @@ def test_read_events(benchmark, num_events):
     provider.add_span_processor(_EventsReadingProcessor())
     tp = provider.get_tracer("bench")
 
+    peaks = []
+    for _ in range(200):
+        span = tp.start_span("benchmarkedSpan")
+        for event in range(num_events):
+            span.add_event(f"event{event}", {"k": "v"})
+        tracemalloc.start()
+        span.end()
+        _, peak = tracemalloc.get_traced_memory()
+        tracemalloc.stop()
+        peaks.append(peak)
+    benchmark.extra_info["mean_alloc_bytes"] = sum(peaks) / len(peaks)
+
     def benchmark_read_events():
         span = tp.start_span("benchmarkedSpan")
         for event in range(num_events):
@@ -187,6 +199,18 @@ def test_read_links(benchmark, num_links):
     provider.add_span_processor(_LinksReadingProcessor())
     tp = provider.get_tracer("bench")
 
+    peaks = []
+    for _ in range(200):
+        span = tp.start_span("benchmarkedSpan")
+        for _ in range(num_links):
+            span.add_link(_link_context)
+        tracemalloc.start()
+        span.end()
+        _, peak = tracemalloc.get_traced_memory()
+        tracemalloc.stop()
+        peaks.append(peak)
+    benchmark.extra_info["mean_alloc_bytes"] = sum(peaks) / len(peaks)
+
     def benchmark_read_links():
         span = tp.start_span("benchmarkedSpan")
         for _ in range(num_links):

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

@@ -63,7 +63,7 @@ def __len__(self):
 
     def __iter__(self):
         with self._lock:
-            return iter(deque(self._dq))
+            return iter(list(self._dq))
 
     def append(self, item):
         with self._lock: