Patches item #1641790, was opened at 2007-01-22 09:00 Message generated for change (Comment added) made by nnorwitz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1641790&group_id=5470
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Library (Lib) Group: Python 2.6 Status: Open Resolution: None Priority: 5 Private: No Submitted By: TH (therve) >Assigned to: Vinay Sajip (vsajip) Summary: logging leaks loggers Initial Comment: In our application, we used to create a logger per client (to get IP/port automatically in the prefix). Unfortunately logging leaks loggers by keeping it into an internal dict (attribute loggerDict of Manager). Attached a patch using a weakref object, with a test. ---------------------------------------------------------------------- >Comment By: Neal Norwitz (nnorwitz) Date: 2007-01-22 20:47 Message: Logged In: YES user_id=33168 Originator: NO Vinay, can you provide some direction? Thanks. ---------------------------------------------------------------------- Comment By: TH (therve) Date: 2007-01-22 09:09 Message: Logged In: YES user_id=1038797 Originator: YES Looking at the documentation, it seems keeping it is mandatory because you must get the same instance with getLogger. Maybe it'd need a documented way to remove from the dict, though. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1641790&group_id=5470 _______________________________________________ Patches mailing list Patches@python.org http://mail.python.org/mailman/listinfo/patches
