dlmarion commented on code in PR #5719:
URL: https://github.com/apache/accumulo/pull/5719#discussion_r2192279049


##########
core/src/main/java/org/apache/accumulo/core/trace/TraceUtil.java:
##########
@@ -96,7 +97,10 @@ public static Span startServerRpcSpan(Class<?> caller, 
String spanName, TInfo ti
 
   private static Span startSpan(Class<?> caller, String spanName, SpanKind 
kind,
       Map<String,String> attributes, TInfo tinfo) {
-    if (!enabled && !Span.current().getSpanContext().isValid()) {
+    Context context = tinfo == null ? null : getContext(tinfo);

Review Comment:
   Are you seeing `getContext` return null? Looking at the OpenTelemetry code, 
it doesn't look like it returns null, but I could be looking at the wrong 
version.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to