Browse files

Reduced default max number of Jetty threads to 50

Cloud environments like Heroku tend to have limit how many native threads
you can have per process. Setting a lower default value means people won't
be caught out as easily.
  • Loading branch information...
1 parent 4c849e6 commit 3bdf0d39b5a202904577db8dbf3724bb3d7185f1 @weavejester weavejester committed Mar 4, 2012
Showing with 2 additions and 2 deletions.
  1. +2 −2 ring-jetty-adapter/src/ring/adapter/jetty.clj
@@ -72,7 +72,7 @@
:key-password - the password to the keystore
:truststore - a truststore to use for SSL connections
:trust-password - the password to the truststore
- :max-threads - the maximum number of threads to use (default 250)
+ :max-threads - the maximum number of threads to use (default 50)
:client-auth - SSL client certificate authenticate, may be set to :need,
:want or :none (defaults to :none)"
[handler options]
@@ -81,7 +81,7 @@
(configurator s))
(doto s
(.setHandler (proxy-handler handler))
- (.setThreadPool (QueuedThreadPool. (options :max-threads 250)))
+ (.setThreadPool (QueuedThreadPool. (options :max-threads 50)))
(when (:join? options true)
(.join s))

0 comments on commit 3bdf0d3

Please sign in to comment.