[FIXED JENKINS-16474]

Based on the feedback from several users, easing this up a little bit, and also reducing the timeout since we see thread dumps where most threads are blocked at the parseURI line.
jenkinsci · Feb 12, 2013 · 3079a06 · jglick · Feb 13, 2013 · kohsuke
1 parent ffe79e8
commit 3079a06
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/java/winstone/HttpListener.java b/src/java/winstone/HttpListener.java
@@ -35,7 +35,7 @@ public class HttpListener implements Listener, Runnable {
     protected static int CONNECTION_TIMEOUT = 60000;
     protected static int BACKLOG_COUNT = 5000;
     protected static boolean DEFAULT_HNL = false;
-    protected static int KEEP_ALIVE_TIMEOUT = 10000;
+    protected static int KEEP_ALIVE_TIMEOUT = 5000;
     protected static int KEEP_ALIVE_SLEEP = 20;
     protected static int KEEP_ALIVE_SLEEP_MAX = 500;
     protected HostGroup hostGroup;

diff --git a/src/java/winstone/cmdline/Option.java b/src/java/winstone/cmdline/Option.java
@@ -87,7 +87,7 @@ public static List<Option<?>> all(Class<?> clazz) {
      * and make everyone slow (or worst case choke every request by OOME), so better to err
      * on the conservative side (and have inbound connections wait in the queue)
      */
-    public static final OInt HANDLER_COUNT_MAX     =integer("handlerCountMax",20);
+    public static final OInt HANDLER_COUNT_MAX     =integer("handlerCountMax",40);
     /**
      * Leave this number of request handler threads in the pool even when they are idle.
      * Other threads are destroyed when they are idle to free up resources.