Ensure exported spans are unsampled + set default anomaly condition usage logic

majanjua-amzn · majanjua-amzn · commit 2c68700bebb1 · 2025-08-22T09:37:36.000-07:00
diff --git a/aws-xray/src/main/java/io/opentelemetry/contrib/awsxray/XrayRulesSampler.java b/aws-xray/src/main/java/io/opentelemetry/contrib/awsxray/XrayRulesSampler.java
@@ -245,6 +245,9 @@ void adaptSampling(ReadableSpan span, SpanData spanData, Consumer<ReadableSpan>
                 shouldCaptureAnomalySpan = true;
                 break;
             }
+          } else {
+            shouldBoostSampling = true;
+            shouldCaptureAnomalySpan = true;
           }
         }
         if (shouldBoostSampling && shouldCaptureAnomalySpan) {
@@ -300,6 +303,7 @@ void adaptSampling(ReadableSpan span, SpanData spanData, Consumer<ReadableSpan>
       }
 
       if (shouldBoostSampling
+          && !span.getSpanContext().isSampled()
           && ruleToReportTo != null
           && ruleToReportTo.hasBoost()
           && this.anomalyTracesSet.add(traceId)) {

Original file line number	Diff line number	Diff line change
`@@ -245,6 +245,9 @@ void adaptSampling(ReadableSpan span, SpanData spanData, Consumer<ReadableSpan>`
`245`	`245`	`shouldCaptureAnomalySpan = true;`
`246`	`246`	`break;`
`247`	`247`	`}`
	`248`	`+ } else {`
	`249`	`+ shouldBoostSampling = true;`
	`250`	`+ shouldCaptureAnomalySpan = true;`
`248`	`251`	`}`
`249`	`252`	`}`
`250`	`253`	`if (shouldBoostSampling && shouldCaptureAnomalySpan) {`
`@@ -300,6 +303,7 @@ void adaptSampling(ReadableSpan span, SpanData spanData, Consumer<ReadableSpan>`
`300`	`303`	`}`
`301`	`304`
`302`	`305`	`if (shouldBoostSampling`
	`306`	`+ && !span.getSpanContext().isSampled()`
`303`	`307`	`&& ruleToReportTo != null`
`304`	`308`	`&& ruleToReportTo.hasBoost()`
`305`	`309`	`&& this.anomalyTracesSet.add(traceId)) {`