refactor(llmobs): drop redundant hasSessionId ternary on LLMObsContext.attach

jessicagamio · jessicagamio · commit 85731d94afb9 · 2026-05-19T15:58:00.000-07:00
diff --git a/dd-java-agent/agent-llmobs/src/main/java/datadog/trace/llmobs/domain/DDLLMObsSpan.java b/dd-java-agent/agent-llmobs/src/main/java/datadog/trace/llmobs/domain/DDLLMObsSpan.java
@@ -120,7 +120,7 @@ public DDLLMObsSpan(
     }
     span.setTag(LLMOBS_TAG_PREFIX + PARENT_ID_TAG_INTERNAL, parentSpanID);
     // Propagate the effective sessionId to descendant LLMObs spans via the context.
-    scope = LLMObsContext.attach(span.context(), this.hasSessionId ? sessionId : null);
+    scope = LLMObsContext.attach(span.context(), sessionId);
   }
 
   @Override

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ public DDLLMObsSpan(`
`120`	`120`	`}`
`121`	`121`	`span.setTag(LLMOBS_TAG_PREFIX + PARENT_ID_TAG_INTERNAL, parentSpanID);`
`122`	`122`	`// Propagate the effective sessionId to descendant LLMObs spans via the context.`
`123`		`- scope = LLMObsContext.attach(span.context(), this.hasSessionId ? sessionId : null);`
	`123`	`+ scope = LLMObsContext.attach(span.context(), sessionId);`
`124`	`124`	`}`
`125`	`125`
`126`	`126`	`@Override`