Removed option to disable post-processing

Author: ValentinZakharov

dd-trace-api/src/main/java/datadog/trace/api/config/TracerConfig.java

@@ -133,8 +133,6 @@ public final class TracerConfig {
 
   public static final String TRACE_FLUSH_INTERVAL = "trace.flush.interval";
 
-  public static final String TRACE_POST_PROCESSING_ENABLED = "trace.post-processing.enabled";
-
   public static final String TRACE_POST_PROCESSING_TIMEOUT = "trace.post-processing.timeout";
 
   private TracerConfig() {}

dd-trace-core/src/main/java/datadog/trace/common/writer/TraceProcessingWorker.java

@@ -21,7 +21,6 @@
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
 import java.util.function.BooleanSupplier;
-
 import org.jctools.queues.MessagePassingQueue;
 import org.jctools.queues.MpscBlockingConsumerArrayQueue;
 import org.slf4j.Logger;
@@ -305,10 +304,6 @@ protected boolean queuesAreEmpty() {
     }
 
     private void maybeTracePostProcessing(List<DDSpan> trace) {
-      if (!Config.get().isTracePostProcessingEnabled()) {
-        return;
-      }
-
       if (trace == null || trace.isEmpty()) {
         return;
       }
@@ -320,8 +315,7 @@ private void maybeTracePostProcessing(List<DDSpan> trace) {
 
       long timeout = Config.get().getTracePostProcessingTimeout();
       long deadline = System.currentTimeMillis() + timeout;
-      BooleanSupplier timeoutCheck =
-          () -> System.currentTimeMillis() > deadline;
+      BooleanSupplier timeoutCheck = () -> System.currentTimeMillis() > deadline;
 
       if (!tracePostProcessor.process(trace, context, timeoutCheck)) {
         log.debug("Trace post-processing is interrupted due to timeout.");

internal-api/src/main/java/datadog/trace/api/Config.java

@@ -434,7 +434,6 @@
 import static datadog.trace.api.config.TracerConfig.TRACE_PEER_SERVICE_COMPONENT_OVERRIDES;
 import static datadog.trace.api.config.TracerConfig.TRACE_PEER_SERVICE_DEFAULTS_ENABLED;
 import static datadog.trace.api.config.TracerConfig.TRACE_PEER_SERVICE_MAPPING;
-import static datadog.trace.api.config.TracerConfig.TRACE_POST_PROCESSING_ENABLED;
 import static datadog.trace.api.config.TracerConfig.TRACE_POST_PROCESSING_TIMEOUT;
 import static datadog.trace.api.config.TracerConfig.TRACE_PROPAGATION_EXTRACT_FIRST;
 import static datadog.trace.api.config.TracerConfig.TRACE_PROPAGATION_STYLE;
@@ -926,7 +925,6 @@ static class HostNameHolder {
 
   private final boolean axisPromoteResourceName;
   private final float traceFlushIntervalSeconds;
-  private final boolean tracePostProcessingEnabled;
   private final long tracePostProcessingTimeout;
 
   private final boolean telemetryDebugRequestsEnabled;
@@ -2040,9 +2038,6 @@ PROFILING_DATADOG_PROFILER_ENABLED, isDatadogProfilerSafeInCurrentEnvironment())
           "Agentless profiling activated but no api key provided. Profile uploading will likely fail");
     }
 
-    this.tracePostProcessingEnabled =
-        configProvider.getBoolean(TRACE_POST_PROCESSING_ENABLED, true);
-
     this.tracePostProcessingTimeout = configProvider.getLong(TRACE_POST_PROCESSING_TIMEOUT, 1000);
 
     if (isCiVisibilityEnabled()
@@ -2145,10 +2140,6 @@ public float getTraceFlushIntervalSeconds() {
     return traceFlushIntervalSeconds;
   }
 
-  public boolean isTracePostProcessingEnabled() {
-    return tracePostProcessingEnabled;
-  }
-
   public long getTracePostProcessingTimeout() {
     return tracePostProcessingTimeout;
   }