fix(jcache): Check for NoOp span after startChild

adinauer · adinauer · commit 038e7926d53f · 2026-03-10T06:19:06.000+01:00
startChild can return a NoOp span (e.g. when span limit is reached).
Skip instrumentation in that case to avoid unnecessary work.
diff --git a/sentry-jcache/src/main/java/io/sentry/jcache/SentryJCacheWrapper.java b/sentry-jcache/src/main/java/io/sentry/jcache/SentryJCacheWrapper.java
@@ -409,6 +409,9 @@ public Iterator<Entry<K, V>> iterator() {
     spanOptions.setOrigin(TRACE_ORIGIN);
     final String keyString = key != null ? String.valueOf(key) : null;
     final ISpan span = activeSpan.startChild(operation, keyString, spanOptions);
+    if (span.isNoOp()) {
+      return null;
+    }
     if (keyString != null) {
       span.setData(SpanDataConvention.CACHE_KEY_KEY, Arrays.asList(keyString));
     }
@@ -429,6 +432,9 @@ public Iterator<Entry<K, V>> iterator() {
     final SpanOptions spanOptions = new SpanOptions();
     spanOptions.setOrigin(TRACE_ORIGIN);
     final ISpan span = activeSpan.startChild(operation, delegate.getName(), spanOptions);
+    if (span.isNoOp()) {
+      return null;
+    }
     span.setData(
         SpanDataConvention.CACHE_KEY_KEY,
         keys.stream().map(String::valueOf).collect(Collectors.toList()));
