move dropped warning into toSpanData, add trace+span id

moving the warning removes an extra call to toSpanData, thus some overhead. Outside of tests, this function is only called from span processors once
open-telemetry · Jun 18, 2024 · d0819ea · d0819ea
1 parent f69bb8c
commit d0819ea
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 17 deletions.
diff --git a/src/SDK/Trace/Span.php b/src/SDK/Trace/Span.php
@@ -251,15 +251,6 @@ public function end(int $endEpochNanos = null): void
             return;
         }
 
-        $spanData = $this->toSpanData();
-        if ($spanData->getTotalDroppedLinks() || $spanData->getTotalDroppedEvents() || $spanData->getAttributes()->getDroppedAttributesCount()) {
-            self::logWarning('Dropped span attributes, links or events', [
-                'attributes' => $spanData->getAttributes()->getDroppedAttributesCount(),
-                'links' => $spanData->getTotalDroppedLinks(),
-                'events' => $spanData->getTotalDroppedEvents(),
-            ]);
-        }
-
         $this->endEpochNanos = $endEpochNanos ?? Clock::getDefault()->now();
         $this->hasEnded = true;
 
@@ -289,7 +280,7 @@ public function hasEnded(): bool
 
     public function toSpanData(): SpanDataInterface
     {
-        return new ImmutableSpan(
+        $spanData = new ImmutableSpan(
             $this,
             $this->name,
             $this->links,
@@ -301,6 +292,18 @@ public function toSpanData(): SpanDataInterface
             $this->endEpochNanos,
             $this->hasEnded
         );
+
+        if ($spanData->getTotalDroppedLinks() || $spanData->getTotalDroppedEvents() || $spanData->getAttributes()->getDroppedAttributesCount()) {
+            self::logWarning('Dropped span attributes, links or events', [
+                'trace_id' => $spanData->getTraceId(),
+                'span_id' => $spanData->getSpanId(),
+                'attributes' => $spanData->getAttributes()->getDroppedAttributesCount(),
+                'links' => $spanData->getTotalDroppedLinks(),
+                'events' => $spanData->getTotalDroppedEvents(),
+            ]);
+        }
+
+        return $spanData;
     }
 
     /** @inheritDoc */

diff --git a/tests/Unit/SDK/Trace/SpanTest.php b/tests/Unit/SDK/Trace/SpanTest.php
@@ -943,15 +943,18 @@ private function assertSpanData(
 
     private function expectDropped(int $attributes, int $events, int $links): void
     {
-        $this->logWriter->expects($this->once())->method('write')->with(
+        $this->logWriter->expects($this->atLeastOnce())->method('write')->with(
             $this->anything(),
             $this->stringContains('Dropped span attributes'),
-            $this->equalTo([
-                'attributes' => $attributes,
-                'links' => $links,
-                'events' => $events,
-                'source' => 'OpenTelemetry\\SDK\\Trace\\Span',
-            ])
+            $this->callback(function (array $context) use ($attributes, $events, $links) {
+                $this->assertSame($context['attributes'], $attributes);
+                $this->assertSame($context['events'], $events);
+                $this->assertSame($context['links'], $links);
+                $this->assertNotNull($context['trace_id']);
+                $this->assertNotNull($context['span_id']);
+
+                return true;
+            }),
         );
     }
 }