HTTPCLIENT-2388: fix builder default evictor sleepTime min 1 second

Author: sanghwa-min

httpclient5/src/main/java/org/apache/hc/client5/http/impl/async/HttpAsyncClientBuilder.java

@@ -37,6 +37,7 @@
 import java.util.LinkedList;
 import java.util.List;
 import java.util.concurrent.ThreadFactory;
+import java.util.concurrent.TimeUnit;
 import java.util.function.Function;
 import java.util.function.UnaryOperator;
 
@@ -158,6 +159,8 @@
  */
 public class HttpAsyncClientBuilder {
 
+    private static final TimeValue ONE_SECOND = TimeValue.ofSeconds(1L);
+
     private static class RequestInterceptorEntry {
 
         enum Position { FIRST, LAST }
@@ -1094,8 +1097,10 @@ public CloseableHttpAsyncClient build() {
             }
             if (evictExpiredConnections || evictIdleConnections) {
                 if (connManagerCopy instanceof ConnPoolControl) {
+                    TimeValue sleepTime = maxIdleTime.divide(10, TimeUnit.NANOSECONDS);
+                    sleepTime = sleepTime.compareTo(ONE_SECOND) < 0 ? ONE_SECOND : sleepTime;
                     final IdleConnectionEvictor connectionEvictor = new IdleConnectionEvictor((ConnPoolControl<?>) connManagerCopy,
-                            maxIdleTime, maxIdleTime);
+                            sleepTime, maxIdleTime);
                     closeablesCopy.add(connectionEvictor::shutdown);
                     connectionEvictor.start();
                 }

httpclient5/src/main/java/org/apache/hc/client5/http/impl/classic/HttpClientBuilder.java

@@ -37,6 +37,7 @@
 import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
+import java.util.concurrent.TimeUnit;
 import java.util.function.Function;
 import java.util.function.UnaryOperator;
 
@@ -145,6 +146,8 @@
  */
 public class HttpClientBuilder {
 
+    private static final TimeValue ONE_SECOND = TimeValue.ofSeconds(1L);
+
     private static class RequestInterceptorEntry {
 
         enum Position { FIRST, LAST }
@@ -1105,8 +1108,10 @@ public CloseableHttpClient build() {
             }
             if (evictExpiredConnections || evictIdleConnections) {
                 if (connManagerCopy instanceof ConnPoolControl) {
+                    TimeValue sleepTime = maxIdleTime.divide(10, TimeUnit.NANOSECONDS);
+                    sleepTime = sleepTime.compareTo(ONE_SECOND) < 0 ? ONE_SECOND : sleepTime;
                     final IdleConnectionEvictor connectionEvictor = new IdleConnectionEvictor((ConnPoolControl<?>) connManagerCopy,
-                            maxIdleTime, maxIdleTime);
+                            sleepTime, maxIdleTime);
                     closeablesCopy.add(() -> {
                         connectionEvictor.shutdown();
                         try {