[ 
https://issues.apache.org/jira/browse/LOG4NET-421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13946552#comment-13946552
 ] 

Tom Voros commented on LOG4NET-421:
-----------------------------------

Thank you for acknowledging the issue.  I'm optimistic about the solution 
you've offered in LOG4NET-429.  :)

I can't seem to edit the description above but I wanted to clarify that the 
"fix" workaround will not work in all cases.  Only certain appender types 
support the <fix ... /> option and unfortunately RollingFileAppender (the most 
common appender type?) is not one of them.

> Inclusion of UserName in composite properties dictionary has a significant 
> performance impact
> ---------------------------------------------------------------------------------------------
>
>                 Key: LOG4NET-421
>                 URL: https://issues.apache.org/jira/browse/LOG4NET-421
>             Project: Log4net
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.2.13
>         Environment: Issue was observed on Windows Server 2008 R2 with an 
> ASP.NET 4.0 application.
>            Reporter: Tom Voros
>            Assignee: Dominik Psenner
>              Labels: performance
>
> A change made for LOG4NET-205 has the LoggingEvent.CreateCompositeProperties 
> method get the UserName property for every log event.  Since getting the 
> UserName is very slow, this change has a significant impact on performance in 
> log4net.
> In one trace captured with dotTrace on a large web application we found that 
> the get UserName property (called from CreateCompositeProperties) accounted 
> for 34% of all time spent in log4net code.
> Our workaround for the issue has been to "fix" the UserName property by 
> adding <fix value="32" /> to all our appenders.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to