chore(com): Reduce API surface

Author: bric3

communication/src/main/java/datadog/communication/http/ahc/ApacheAsyncHttpClientFactory.java

@@ -1,4 +1,4 @@
-package datadog.communication.http.ahc;
+package datadog.communication.http.client.ahc;
 
 import datadog.communication.http.HttpRetryPolicy;
 import datadog.communication.http.client.HttpClient;

…tion/http/ahc/ApacheAsyncHttpClient.java …tp/client/ahc/ApacheAsyncHttpClient.javacommunication/src/main/java/datadog/communication/http/ahc/ApacheAsyncHttpClient.java renamed to communication/src/main/java/datadog/communication/http/client/ahc/ApacheAsyncHttpClient.java communication/src/main/java/datadog/communication/http/ahc/ApacheAsyncHttpClient.java renamed to communication/src/main/java/datadog/communication/http/client/ahc/ApacheAsyncHttpClient.java

@@ -1,4 +1,4 @@
-package datadog.communication.http.ahc;
+package datadog.communication.http.client.ahc;
 
 import datadog.communication.http.HttpRetryPolicy;
 import datadog.communication.http.client.HttpClient;

…tp/ahc/ApacheAsyncHttpClientBuilder.java …nt/ahc/ApacheAsyncHttpClientBuilder.javacommunication/src/main/java/datadog/communication/http/ahc/ApacheAsyncHttpClientBuilder.java renamed to communication/src/main/java/datadog/communication/http/client/ahc/ApacheAsyncHttpClientBuilder.java communication/src/main/java/datadog/communication/http/ahc/ApacheAsyncHttpClientBuilder.java renamed to communication/src/main/java/datadog/communication/http/client/ahc/ApacheAsyncHttpClientBuilder.java

@@ -1,4 +1,4 @@
-package datadog.communication.http.netty;
+package datadog.communication.http.client.netty;
 
 import datadog.common.socket.NamedPipeSocketFactory;
 import datadog.common.socket.UnixDomainSocketFactory;

…nication/http/netty/NettyHttpClient.java …n/http/client/netty/NettyHttpClient.javacommunication/src/main/java/datadog/communication/http/netty/NettyHttpClient.java renamed to communication/src/main/java/datadog/communication/http/client/netty/NettyHttpClient.java communication/src/main/java/datadog/communication/http/netty/NettyHttpClient.java renamed to communication/src/main/java/datadog/communication/http/client/netty/NettyHttpClient.java

@@ -1,4 +1,4 @@
-package datadog.communication.http.netty;
+package datadog.communication.http.client.netty;
 
 import datadog.communication.http.HttpRetryPolicy;
 import datadog.communication.http.client.HttpClient;

…n/http/netty/NettyHttpClientBuilder.java …client/netty/NettyHttpClientBuilder.javacommunication/src/main/java/datadog/communication/http/netty/NettyHttpClientBuilder.java renamed to communication/src/main/java/datadog/communication/http/client/netty/NettyHttpClientBuilder.java communication/src/main/java/datadog/communication/http/netty/NettyHttpClientBuilder.java renamed to communication/src/main/java/datadog/communication/http/client/netty/NettyHttpClientBuilder.java

@@ -10,9 +10,8 @@
 import static org.junit.jupiter.api.Assumptions.assumeTrue;
 
 import datadog.communication.http.HttpRetryPolicy;
-import datadog.communication.http.ahc.ApacheAsyncHttpClientFactory;
-import datadog.communication.http.netty.NettyHttpClientBuilder;
-import datadog.communication.http.netty.NettyHttpClientFactory;
+import datadog.communication.http.client.ahc.ApacheAsyncHttpClientBuilder;
+import datadog.communication.http.client.netty.NettyHttpClientBuilder;
 import io.netty.channel.nio.NioEventLoopGroup;
 import java.io.Closeable;
 import java.io.IOException;
@@ -277,7 +276,7 @@ void shouldNotCloseExternallyManagedEventLoopGroupWhenSupported() {
 
     NioEventLoopGroup externalGroup = new NioEventLoopGroup(1);
     try (HttpClient ignored =
-        NettyHttpClientFactory.builder().eventLoopGroup(externalGroup, false).build()) {
+        new NettyHttpClientBuilder().eventLoopGroup(externalGroup, false).build()) {
       // no-op
     }
     assertFalse(externalGroup.isShuttingDown());
@@ -333,11 +332,11 @@ public void beforeEach(ExtensionContext context) {
       switch (implementationValue) {
         case NETTY:
           ((HttpClientContractTest) context.getRequiredTestInstance())
-              .useBuilder(NettyHttpClientFactory.builder());
+              .useBuilder(new NettyHttpClientBuilder());
           break;
         case AHC:
           ((HttpClientContractTest) context.getRequiredTestInstance())
-              .useBuilder(ApacheAsyncHttpClientFactory.builder());
+              .useBuilder(new ApacheAsyncHttpClientBuilder());
           break;
         default:
           throw new IllegalStateException("Unsupported implementation: " + implementationValue);