[issue30962] Add caching to logging.Logger.isEnabledFor()

Change by Zane Bitter : -- pull_requests: +15009 pull_request: https://github.com/python/cpython/pull/15286 ___ Python tracker ___ _

[issue30962] Add caching to logging.Logger.isEnabledFor()

Changes by Vinay Sajip : -- resolution: -> fixed stage: -> resolved status: open -> closed ___ Python tracker ___ ___ Python-bugs-li

[issue30962] Add caching to logging.Logger.isEnabledFor()

Vinay Sajip added the comment: New changeset 78c18a9b9a1445f7c755929917a790ba02b4a5e0 by Vinay Sajip (Avram Lubkin) in branch 'master': bpo-30962: Added caching to Logger.isEnabledFor() (GH-2752) https://github.com/python/cpython/commit/78c18a9b9a1445f7c755929917a790ba02b4a5e0 -- ___

[issue30962] Add caching to logging.Logger.isEnabledFor()

STINNER Victor added the comment: >> Benchmark code is in the referenced pull request > Does this speed up actual logging calls such logger.info, etc? According to the benchmark in the PR, it's 50% faster when the log is ignored, like a debug call in production. -- nosy: +haypo __

[issue30962] Add caching to logging.Logger.isEnabledFor()

Avram added the comment: Yes, though the bottleneck for normal logging is more likely in other areas so the speedup will be less noticeable. Where I notice it is with debug statements when debugging is disabled. In that scenario the bulk of the time is spent checking if the level is enabled an

[issue30962] Add caching to logging.Logger.isEnabledFor()

Antoine Pitrou added the comment: Does this speed up actual logging calls such logger.info, etc? -- nosy: +pitrou ___ Python tracker ___ _

[issue30962] Add caching to logging.Logger.isEnabledFor()

Changes by Antoine Pitrou : -- nosy: +vinay.sajip ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.

[issue30962] Add caching to logging.Logger.isEnabledFor()

New submission from Avram: Checking to see if a log level is enabled can add non-insignificant time to programs with a large number of logging statements. This issue is to track a pull request which introduces caching to the logging module so checking to see if a log level is enabled takes hal