Vinay Sajip <vinay_sa...@yahoo.co.uk> added the comment:

Bill,

Thanks for the suggestion and the patch. It's a good idea, though I wonder 
whether you found that the bulk of the time spent in isEnabledFor() was 
actually spent in getEffectiveLevel()? That's the one which loops through a 
logger's ancestors looking for a level which is actually set, so intuitively it 
would take a while - especially for deeper levels in the logging hierarchy. If 
so (which I suspect to be the case, but it would be good to have you confirm 
it), a better solution may be to cache the effective level.

Roughly how deep are/were your logger hierarchies in the situation where you 
experienced performance problems?

I'm happy to look at caching effective level for Python 3.3: The 2.X branches 
are now closed for additions other than bugs and security issues.

----------
assignee:  -> vinay.sajip

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue11369>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to