Merge pull request #5192 from getsentry/fix/cache-tracing-noop-span

fix(spring): [Cache Tracing 11] Skip cache span data when child span is NoOp

Author: adinauer

sentry-spring-7/src/main/java/io/sentry/spring7/cache/SentryCacheWrapper.java

@@ -295,6 +295,9 @@ public boolean invalidate() {
     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));
     }

sentry-spring-jakarta/src/main/java/io/sentry/spring/jakarta/cache/SentryCacheWrapper.java

@@ -295,6 +295,9 @@ public boolean invalidate() {
     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));
     }

sentry-spring/src/main/java/io/sentry/spring/cache/SentryCacheWrapper.java

@@ -223,6 +223,9 @@ public boolean invalidate() {
     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));
     }