Mike
Todd Wolff wrote:
Sure, the code is yours to do with as you wish. Related to the performance issue, I am still seeing a degradation by a factor of roughly 2.5. - and yes, logging is set to error level :-)
The problem may be isolated to httpclient's interaction with the Jetty container. Oleg, I recall you saying that you were using the Tomcat distribution. My results using 10 threads:
JBoss-3.0.4(Jetty) alpha3 50 messages in .036 minutes beta2 50 messages in .084
JBoss - 3.2.1(Jetty) alpha3 50 messages in .035 minutes beta2 50 messages in .083
----- Original Message ----- From: "Kalnichevski, Oleg" <[EMAIL PROTECTED]> To: "Commons HttpClient Project" <[EMAIL PROTECTED]> Sent: Tuesday, July 29, 2003 5:55 AM Subject: RE: Performance Issue
Eric, All credits should go to Todd. He's done pretty much all the hard work.
Maybe it is just me, but I can live with a 1ms penalty that dramatically increases the reliability of the re-used connections. Based on your research, I think we should keep the isStale() check. What do others
think?
I agree. I think this performance penalty is well justified. I would still like to have an option of disabling the check (especially for the single-threaded connection manager), though, as the 'stale' connection check also has some nasty side-effects on SSL connections when running on older JDKs.
You might consider committing your performance test as something under the "contrib" package so that we could look at running it with each release, and thus keep track of the library's performance over time.
Todd, would you mind contributing your code to the project?
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]
