hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 35944] - Connection is not released back to the pool if a runtime exception is thrown in HttpMethod#releaseConnection method
Date Fri, 05 Aug 2005 13:43:59 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35944>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35944





------- Additional Comments From olegk@apache.org  2005-08-05 15:43 -------
(In reply to comment #17)
> It's possible to catch the IllegalStateException instead. That would certainly
> be faster:
> 

I seriuosly doubt that a boolean variable test would be slower than setting up a
try/catch/finally frame.

Symantically (at least IMO) IllegalStateException should represent a fatal
exception one should not attempt to recover from. I would rather not throw it
all (see my original patch)

Oleg

> boolean avail;
> try {
>   avail = responseConnection.isResponseAvailable();
> } catch(IllegalStateException e) {
>   avail = false;
> }
> 
> if(avail) { ...



-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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


Mime
View raw message