Mailing-List: contact issues-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
Date: Wed, 7 Sep 2016 07:46:20 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: issues@camel.apache.org
Message-ID: <JIRA.12996749.1470946924000.505411.1473234380588@Atlassian.JIRA>
In-Reply-To: <JIRA.12996749.1470946924000@Atlassian.JIRA>
References: <JIRA.12996749.1470946924000@Atlassian.JIRA> <JIRA.12996749.1470946924065@arcas>
Subject: [jira] [Commented] (CAMEL-10240) CamelHttpClient default pool size
 does not work on system with large number of cpus
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Wed, 07 Sep 2016 07:46:22 -0000


    [ https://issues.apache.org/jira/browse/CAMEL-10240?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15469899#comment-15469899 ] 

ASF GitHub Bot commented on CAMEL-10240:
----------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/camel/pull/1157


> CamelHttpClient default pool size does not work on system with large number of cpus
> -----------------------------------------------------------------------------------
>
>                 Key: CAMEL-10240
>                 URL: https://issues.apache.org/jira/browse/CAMEL-10240
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-jetty
>    Affects Versions: 2.17.2
>            Reporter: Zoltan Farkas
>
> In CamelHttpClient a default threadpool is being created with a hardcoded size of 16:
> {code}
>     @Override
>     protected void doStart() throws Exception {
>         if (!hasThreadPool()) {
>             // if there is no thread pool then create a default thread pool using daemon threads
>             QueuedThreadPool qtp = new QueuedThreadPool();
>             // 16 max threads is the default in the http client
>             qtp.setMaxThreads(16);
>             qtp.setDaemon(true);
>             // let the thread names indicate they are from the client
>             qtp.setName("CamelJettyClient(" + ObjectHelper.getIdentityHashCode(this) + ")");
>             setThreadPoolOrExecutor(qtp);
>         }
>         super.doStart();
>     }
> {code}
> this configuration does not work because number of selectors is calculated relative to the number of CPUs in HttpClientTransportOverHTTP:
> {code}
>    public HttpClientTransportOverHTTP()
>     {
>         this(Math.max(1, Runtime.getRuntime().availableProcessors() / 2));
>     }
> {code}
> As a result by default camel simply hangs on these systems...
> It would be nice if CamelHttpClient would have a constructor that set the HttpClientTransport or at least the number of selectors.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)