On Tue, Nov 01, 2011 at 10:09:43AM -0400, Bill Speirs wrote:
> > Good ol' #releaseConnection() method will be back in HttpClient 4.2
> 
> Good. I think that will help to simplify things and cause less
> confusion. I don't know... maybe I'm the only confused one :-)
> 
> > This code is likely to leak connections in case of a runtime (unchecked) 
> > exception.
> 
> Good point...
> 
> While we're discussing it, why are there separate classes/interfaces
> in core and client for an HttpRequest:
> 
> org.apache.http.HttpRequest - core
> org.apache.http.client.methods.HttpRequestBase - client
> 
> Would it be beneficial to merge some of this together? Is it that the
> classes in core simply represent a request as it arrives at a server,
> and the methods from client represent the request as it's being made
> to the server. Aren't they about the same?
> 

I personally think they are not. For instance, #abort and #releaseConnection 
methods make no sense in the server context. The whole concept of connection 
release is simply not applicable to the server side. At least that is how I see 
it. 

Oleg

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to