Update TraceTranslator.java (#296)

WadeGulbrandsen · psx95 · web-flow · commit d1f8b662e0c0 · 2024-02-12T14:52:34.000-05:00
Check to see if parent context is valid instead of parent span ID not null

Co-authored-by: Pranav Sharma &lt;sharmapranav@google.com&gt;
diff --git a/exporters/trace/src/main/java/com/google/cloud/opentelemetry/trace/TraceTranslator.java b/exporters/trace/src/main/java/com/google/cloud/opentelemetry/trace/TraceTranslator.java
@@ -120,7 +120,7 @@ Span generateSpan(SpanData spanData, String projectId) {
       spanBuilder.setEndTime(toTimestampProto(end));
     }
     spanBuilder.setLinks(toLinksProto(spanData.getLinks(), spanData.getTotalRecordedLinks()));
-    if (spanData.getParentSpanId() != null) {
+    if (spanData.getParentSpanContext().isValid()) {
       spanBuilder.setParentSpanId(spanData.getParentSpanId());
     }
     boolean hasRemoteParent = spanData.getParentSpanContext().isRemote();

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ Span generateSpan(SpanData spanData, String projectId) {`
`120`	`120`	`spanBuilder.setEndTime(toTimestampProto(end));`
`121`	`121`	`}`
`122`	`122`	`spanBuilder.setLinks(toLinksProto(spanData.getLinks(), spanData.getTotalRecordedLinks()));`
`123`		`- if (spanData.getParentSpanId() != null) {`
	`123`	`+ if (spanData.getParentSpanContext().isValid()) {`
`124`	`124`	`spanBuilder.setParentSpanId(spanData.getParentSpanId());`
`125`	`125`	`}`
`126`	`126`	`boolean hasRemoteParent = spanData.getParentSpanContext().isRemote();`