extract method

Author: trask

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/init/SecondEntryPoint.java

@@ -260,25 +260,6 @@ public void customize(AutoConfigurationCustomizer autoConfiguration) {
               return props;
             })
         .addPropertiesCustomizer(new AiConfigCustomizer())
-        .addLogRecordProcessorCustomizer(
-            (logRecordProcessor, configProperties) -> {
-              if (logRecordProcessor instanceof BatchLogRecordProcessor) {
-                List<LogRecordProcessor> logRecordProcessors =
-                    getLogRecordProcessors(configuration);
-
-                // the filtering log record processor needs to be chained on front of the batch log
-                // record processor, hopefully log filtering will be better supported by
-                // OpenTelemetry SDK in the future, see
-                // https://github.com/open-telemetry/opentelemetry-specification/pull/4439
-                logFilteringProcessor =
-                    createLogFilteringProcessor(logRecordProcessor, configuration);
-
-                logRecordProcessors.add(logFilteringProcessor);
-                return LogRecordProcessor.composite(
-                    logRecordProcessors.toArray(new LogRecordProcessor[0]));
-              }
-              return logRecordProcessor;
-            })
         .addSpanExporterCustomizer(
             (spanExporter, configProperties) -> {
               if (spanExporter instanceof AzureMonitorSpanExporterProvider.MarkerSpanExporter) {
@@ -295,6 +276,13 @@ public void customize(AutoConfigurationCustomizer autoConfiguration) {
                 return metricExporter;
               }
             })
+        .addLogRecordProcessorCustomizer(
+            (logRecordProcessor, configProperties) -> {
+              if (logRecordProcessor instanceof BatchLogRecordProcessor) {
+                return wrapBatchLogRecordProcessor(logRecordProcessor, configuration);
+              }
+              return logRecordProcessor;
+            })
         .addLogRecordExporterCustomizer(
             (logRecordExporter, configProperties) -> {
               if (logRecordExporter
@@ -322,6 +310,20 @@ public void customize(AutoConfigurationCustomizer autoConfiguration) {
         });
   }
 
+  private static LogRecordProcessor wrapBatchLogRecordProcessor(
+      LogRecordProcessor logRecordProcessor, Configuration configuration) {
+    List<LogRecordProcessor> logRecordProcessors = getLogRecordProcessors(configuration);
+
+    // the filtering log record processor needs to be chained on front of the batch log
+    // record processor, hopefully log filtering will be better supported by
+    // OpenTelemetry SDK in the future, see
+    // https://github.com/open-telemetry/opentelemetry-specification/pull/4439
+    logFilteringProcessor = createLogFilteringProcessor(logRecordProcessor, configuration);
+
+    logRecordProcessors.add(logFilteringProcessor);
+    return LogRecordProcessor.composite(logRecordProcessors.toArray(new LogRecordProcessor[0]));
+  }
+
   private static AzureMonitorLogFilteringProcessor createLogFilteringProcessor(
       LogRecordProcessor logRecordProcessor, Configuration configuration) {