Support spans that are split into multiple batches (#3539)

* Support spans across batches * changelog
getsentry · Jul 2, 2024 · a62056e · a62056e
1 parent c7232fe
commit a62056e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,8 @@
 
 ### Fixes
 
+- Support spans that are split into multiple batches ([#3539](https://github.com/getsentry/sentry-java/pull/3539))
+  - When spans belonging to a single transaction were split into multiple batches for SpanExporter, we did not add all spans because the isSpanTooOld check wasn't inverted.
 - Parse and use `send-default-pii` and `max-request-body-size` from `sentry.properties` ([#3534](https://github.com/getsentry/sentry-java/pull/3534))
 
 ## 8.0.0-alpha.2

diff --git a/...y/sentry-opentelemetry-core/src/main/java/io/sentry/opentelemetry/SentrySpanExporter.java b/...y/sentry-opentelemetry-core/src/main/java/io/sentry/opentelemetry/SentrySpanExporter.java
@@ -105,7 +105,8 @@ public CompletableResultCode export(Collection<SpanData> spans) {
     final @NotNull SentryInstantDate now = new SentryInstantDate();
 
     final @NotNull List<SpanData> nonExpired =
-        remaining.stream().filter((span) -> isSpanTooOld(span, now)).collect(Collectors.toList());
+        remaining.stream().filter((span) -> !isSpanTooOld(span, now)).collect(Collectors.toList());
+
     this.finishedSpans.addAll(nonExpired);
 
     // TODO