Generally speaking I agree. I wanted to look at it more carefully though to verify that a method that throws an exception has no case for holding on to the connection. If you cannot think of any reason, then that's good enough for me.
Mike
Oleg Kalnichevski wrote:
Mike, Are there any reasons for NOT automatically releasing connection? I can't think of any. We should still require HttpMethod#releaseConnection() to be called explicitly regardless if there was an exception or not, but in those cases where we can be sure that the connection is not needed anymore, we should do it automatically.
Oleg
On Tue, 2003-10-07 at 17:41, Michael Becke wrote:
Yes, a final decision still needs to be made on that one. Do you have any thoughts?
Mike
Kalnichevski, Oleg wrote:
Mike, What is the outcome of this one? Is there anything we may need to do regarding automatic connection release in abnormal situations
http://marc.theaimsgroup.com/?t=106442705600001&r=1&w=2
Oleg
-----Original Message----- From: Michael Becke [mailto:[EMAIL PROTECTED] Sent: Tuesday, October 07, 2003 14:17 To: Commons HttpClient Project Subject: Re: Is there still anything that blocks 2.0RC2 release?
Not that I know of.
Mike
On Tuesday, October 7, 2003, at 04:33 AM, Kalnichevski, Oleg wrote:
Is there still anything that blocks 2.0RC2 release?
Oleg
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
