Mailing-List: contact dev-help@hc.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "HttpComponents Project" <dev@hc.apache.org>
Received-SPF: neutral (nike.apache.org: local policy)
Message-ID: <4A36B801.90305@apache.org>
Date: Mon, 15 Jun 2009 23:07:13 +0200
From: Oleg Kalnichevski <olegk@apache.org>
User-Agent: Thunderbird 2.0.0.21 (Windows/20090302)
MIME-Version: 1.0
To: HttpComponents Project <dev@hc.apache.org>
Subject: Re: Question on HTTPClient-675
References: <f97c4b410906150957k107ccc3s79fd475ad3f4afe5@mail.gmail.com>
 <4A368361.9090405@odi.ch>
In-Reply-To: <4A368361.9090405@odi.ch>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit

Ortwin Gl�ck wrote:
> David,
> 
>> We are still using HttpClient 3.0.1
> 
> Consider upgrading.
> 
>> I saw there is a bug HTTPClient-675 that was fixed by Roland Weber in
>> version 3.1.  It talks about thread-starving in the
>> MultiThreadedHttpConnectionManager and made me wonder if we were seeing
>> something similar occurring.  
> 
> As you are not using the MTCM, that issue does not affect you.
> 
>> I emailed Roland
> 
> Generally you do not email indidivual developers. They might not even bother
> responding.
> 
>>         HttpClient client = new HttpClient();
>>         PostMethod post = new PostMethod(inUrl);
>>         post.setRequestEntity(new StringRequestEntity(inPayload, null,
>> null));
>>         int statusCode = client.executeMethod(post);
>>         log.debug(new StringBuilder("inUrl=").append(inUrl).append(",
>> inPayload=")
>>                     .append(inPayload).append(", statusCode=")
>>                     .append(statusCode).toString());
>>         String payload = readPayload(post);
>>         return payload;
>>
>> The readPayload method just uses post.getResponseBodyAsStream and a
>> BufferedReader to read the response.
>> The BufferedReader is then closed in a finally block.
> 
> This code does not call post.releaseConnection() at all! It will leak
> connections. It could be that you are running out of file descriptors. The call
> should be put into a finally block. See the example code on the website.
> 

David,

Another big issue is that a new instance of HttpClient is created for 
each new request. This is enormously wasteful, as every time there 
likely to be an open connection going out of scope, which needs to be 
GC-ed in order to free up system resources.

You should re-use HttpClient instance for multiple request and shut down 
its connection manager when it is no longer used.

Besides, upgrading to Httpclient 4.0 may be a big jump, but there is no 
excuse for not upgrading to HttpClient 3.1

Oleg


>> Sincerely,
>> David Hamilton
> 
> Hope that helps,
> 
> Ortwin
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org