Fixing redundant code in OtlpUdpSpanExporterBuilder file to make a private cmethod and reuse it again wherever needed

Author: Jeel Mehta

exporters/aws-otel-otlp-udp-exporter/build.gradle.kts

@@ -27,10 +27,10 @@ dependencies {
   compileOnly("io.opentelemetry:opentelemetry-api")
   compileOnly("io.opentelemetry:opentelemetry-sdk")
   compileOnly("io.opentelemetry:opentelemetry-exporter-otlp-common")
+  compileOnly("com.google.code.findbugs:jsr305:3.0.2")
   testImplementation("io.opentelemetry:opentelemetry-api")
   testImplementation("io.opentelemetry:opentelemetry-sdk")
   testImplementation("io.opentelemetry:opentelemetry-exporter-otlp-common")
-  compileOnly("com.google.code.findbugs:jsr305:3.0.2")
   testImplementation(platform("org.junit:junit-bom:5.9.2"))
   testImplementation("org.junit.jupiter:junit-jupiter-api")
   testImplementation("org.junit.jupiter:junit-jupiter-engine")

exporters/aws-otel-otlp-udp-exporter/src/main/java/software/amazon/opentelemetry/exporters/otlp/udp/trace/OtlpUdpSpanExporterBuilder.java

@@ -44,10 +44,7 @@ public final class OtlpUdpSpanExporterBuilder {
   public OtlpUdpSpanExporterBuilder setEndpoint(String endpoint) {
     requireNonNull(endpoint, "endpoint must not be null");
     try {
-      String[] parts = endpoint.split(":");
-      String host = parts[0];
-      int port = Integer.parseInt(parts[1]);
-      this.sender = new UdpSender(host, port);
+      this.sender = createSenderFromEndpoint(endpoint);
     } catch (Exception e) {
       throw new IllegalArgumentException("Invalid endpoint, must be a valid URL: " + endpoint, e);
     }
@@ -62,8 +59,15 @@ public OtlpUdpSpanExporterBuilder setPayloadSampleDecision(TracePayloadSampleDec
     return this;
   }
 
+  private UdpSender createSenderFromEndpoint(String endpoint) {
+        String[] parts = endpoint.split(":");
+        String host = parts[0];
+        int port = Integer.parseInt(parts[1]);
+        return new UdpSender(host, port);
+    }
+
   // For testing purposes
-  public OtlpUdpSpanExporterBuilder withEnvironmentVariables(Map<String, String> env) {
+  OtlpUdpSpanExporterBuilder withEnvironmentVariables(Map<String, String> env) {
     this.environmentVariables = env;
     return this;
   }
@@ -82,10 +86,7 @@ public OtlpUdpSpanExporter build() {
         endpoint = environmentVariables.get(AWS_XRAY_DAEMON_ADDRESS_CONFIG);
         if (endpoint != null && !endpoint.isEmpty()) {
           try {
-            String[] parts = endpoint.split(":");
-            String host = parts[0];
-            int port = Integer.parseInt(parts[1]);
-            this.sender = new UdpSender(host, port);
+            this.sender = createSenderFromEndpoint(endpoint);
             return new OtlpUdpSpanExporter(
                 this.sender, PROTOCOL_HEADER + PROTOCOL_DELIMITER + tracePayloadPrefix);
           } catch (Exception e) {