pr review

Author: zeitlinger

instrumentation/internal/internal-application-logger/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/internal/logging/ApplicationLoggingInstrumentationModule.java

@@ -8,12 +8,9 @@
 import static java.util.Arrays.asList;
 
 import com.google.auto.service.AutoService;
-import io.opentelemetry.api.GlobalOpenTelemetry;
-import io.opentelemetry.instrumentation.api.incubator.config.internal.DeclarativeConfigUtil;
 import io.opentelemetry.javaagent.extension.instrumentation.InstrumentationModule;
 import io.opentelemetry.javaagent.extension.instrumentation.TypeInstrumentation;
 import io.opentelemetry.javaagent.tooling.config.EarlyInitAgentConfig;
-import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
 import java.util.List;
 
 @AutoService(InstrumentationModule.class)

testing/agent-exporter/src/main/java/io/opentelemetry/javaagent/testing/instrumentation/DenyUnsafeInstrumentationModule.java

@@ -34,12 +34,6 @@ public List<TypeInstrumentation> typeInstrumentations() {
 
   @Override
   public boolean defaultEnabled() {
-    // This setting is for testing purposes only.
-    // We're using a system property here will enable the instrumentation even when declarative
-    // config is used.
-    // Otherwise, we'd need to patch YAML files to enable this instrumentation.
-    // It's still possible to enable/disable it via declarative config, this just controls the
-    // default.
     return Boolean.getBoolean("otel.javaagent.testing.deny-unsafe");
   }
 }