remove deprecations

- methods in CtAsyncHttpClient - methods in RetryMiddleware - ApiFactory - ImportApiFactory - MLApiRootFactory - HistoryApiFactory - "rmf:okhttp-client: module - ClientFactory - MiddlewareFactory - ExceptionFactory - VrapJsonUtils
commercetools · Oct 13, 2023 · 77dbb2c · 77dbb2c
1 parent 76f7e8d
commit 77dbb2c
Show file tree

Hide file tree

Showing 19 changed files with 5 additions and 1,411 deletions.
diff --git a/build.gradle b/build.gradle
@@ -204,7 +204,6 @@ def documentationProjects= [
         ":commercetools:commercetools-sdk-java-importapi",
         ":commercetools:commercetools-sdk-java-ml",
         ":commercetools:internal-docs",
-        ":rmf:okhttp-client",
         ":rmf:rmf-java-base"
 ]
 

diff --git a/...s-async-http-client/src/main/java/com/commercetools/http/asynchttp/CtAsyncHttpClient.java b/...s-async-http-client/src/main/java/com/commercetools/http/asynchttp/CtAsyncHttpClient.java
@@ -36,18 +36,6 @@ public CtAsyncHttpClient() {
             clientBuilder.get().addRequestFilter(new ThrottleRequestFilter(MAX_REQUESTS)).build());
     }
 
-    /**
-     * @deprecated usage of maxConnTotal and maxConnPerRoute not advised as there is no pooling done by AHC. Use {@link CtAsyncHttpClient#CtAsyncHttpClient(int)} instead.
-     * @param maxConnPerRoute maximum number of parallel connections to one host
-     * @param maxConnTotal maximum number of parallel connections
-     */
-    @Deprecated
-    public CtAsyncHttpClient(final int maxConnTotal, final int maxConnPerRoute) {
-        super();
-        asyncHttpClient = new DefaultAsyncHttpClient(
-            clientBuilder.get().setMaxConnections(maxConnTotal).setMaxConnectionsPerHost(maxConnPerRoute).build());
-    }
-
     /**
      * Uses a {@link ThrottleRequestFilter} to limit the number of concurrent requests
      * @param maxConnections maximum number of parallel connections
@@ -67,20 +55,6 @@ public CtAsyncHttpClient(final BuilderOptions options) {
         asyncHttpClient = new DefaultAsyncHttpClient(options.plus(clientBuilder.get()).build());
     }
 
-    /**
-     * @deprecated usage of maxConnTotal and maxConnPerRoute not advised as there is no pooling done by AHC. Use {@link CtAsyncHttpClient#CtAsyncHttpClient(int, BuilderOptions)} instead.
-     * @param maxConnPerRoute maximum number of parallel connections to one host
-     * @param maxConnTotal maximum number of parallel connections
-     * @param options options to build the underlying HTTP client
-     */
-    @Deprecated
-    public CtAsyncHttpClient(final int maxConnTotal, final int maxConnPerRoute, final BuilderOptions options) {
-        super();
-        asyncHttpClient = new DefaultAsyncHttpClient(
-            options.plus(clientBuilder.get().setMaxConnections(maxConnTotal).setMaxConnectionsPerHost(maxConnPerRoute))
-                    .build());
-    }
-
     /**
      * Uses a {@link ThrottleRequestFilter} to limit the number of concurrent requests
      * @param maxConnections maximum number of parallel connections
@@ -110,19 +84,6 @@ public CtAsyncHttpClient(final ExecutorService executor) {
         asyncHttpClient = new DefaultAsyncHttpClient(clientBuilder.get().build());
     }
 
-    /**
-     * @deprecated usage of maxConnTotal and maxConnPerRoute not advised as there is no pooling done by AHC. Use {@link CtAsyncHttpClient#CtAsyncHttpClient(ExecutorService, int)} instead.
-     * @param maxConnPerRoute maximum number of parallel connections to one host
-     * @param maxConnTotal maximum number of parallel connections
-     * @param executor ExecutorService to be used
-     */
-    @Deprecated
-    public CtAsyncHttpClient(final ExecutorService executor, final int maxConnTotal, final int maxConnPerRoute) {
-        super(executor);
-        asyncHttpClient = new DefaultAsyncHttpClient(
-            clientBuilder.get().setMaxConnections(maxConnTotal).setMaxConnectionsPerHost(maxConnPerRoute).build());
-    }
-
     /**
      * Uses a {@link ThrottleRequestFilter} to limit the number of concurrent requests
      * @param maxConnections maximum number of parallel connections
@@ -144,22 +105,6 @@ public CtAsyncHttpClient(final ExecutorService executor, final BuilderOptions op
         asyncHttpClient = new DefaultAsyncHttpClient(options.plus(clientBuilder.get()).build());
     }
 
-    /**
-     * @deprecated usage of maxConnTotal and maxConnPerRoute not advised as there is no pooling done by AHC. Use {@link CtAsyncHttpClient#CtAsyncHttpClient(ExecutorService, int, BuilderOptions)} instead.
-     * @param maxConnPerRoute maximum number of parallel connections to one host
-     * @param maxConnTotal maximum number of parallel connections
-     * @param executor ExecutorService to be used
-     * @param options options to build the underlying HTTP client
-     */
-    @Deprecated
-    public CtAsyncHttpClient(final ExecutorService executor, final int maxConnTotal, final int maxConnPerRoute,
-            final BuilderOptions options) {
-        super(executor);
-        asyncHttpClient = new DefaultAsyncHttpClient(
-            options.plus(clientBuilder.get().setMaxConnections(maxConnTotal).setMaxConnectionsPerHost(maxConnPerRoute))
-                    .build());
-    }
-
     /**
      * Uses a {@link ThrottleRequestFilter} to limit the number of concurrent requests
      * @param maxConnections maximum number of parallel connections

diff --git a/...mercetools-sdk-java-api/src/main/java/com/commercetools/api/defaultconfig/ApiFactory.java b/...mercetools-sdk-java-api/src/main/java/com/commercetools/api/defaultconfig/ApiFactory.java