hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tony Poppleton (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HTTPCLIENT-895) Log creation impairs performance
Date Mon, 30 Nov 2009 20:48:20 GMT

    [ https://issues.apache.org/jira/browse/HTTPCLIENT-895?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12783838#action_12783838

Tony Poppleton commented on HTTPCLIENT-895:

The second solution would also mean that there is a risk this bug might resurface in the future,
either in the same class or in another part of the library by the introduction of a Log. 
So the "static" solution would be better in this respect.

> Log creation impairs performance
> --------------------------------
>                 Key: HTTPCLIENT-895
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-895
>             Project: HttpComponents HttpClient
>          Issue Type: Improvement
>          Components: HttpClient
>    Affects Versions: 4.0 Final
>            Reporter: Tony Poppleton
>            Priority: Minor
> Running JProfiler on a program that uses HttpClient with a ThreadSafeClientConnManager,
revealed that 5% of the time was spent constructing Log instances in class ClientParamsStack.
> Oleg did some further investigation and found that DefaultRequestDirector also has the
same problem.
> A simple solution would be to make the Log a static member variable, and do this on all
classes for consistency.  However this might not be the best solution for interoperating with
some frameworks (see http://wiki.apache.org/jakarta-commons/Logging/StaticLog)
> Another solution would be to simply remove the Log from the affected classes, although
they are presumably there for a reason...

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

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

View raw message