LucidDB / luciddb

DEFUNCT: See README
https://github.com/LucidDB/luciddb
Apache License 2.0
53 stars 24 forks source link

[FRG-309] generalize leak prevention for native tracing #563

Open dynamobi-build opened 12 years ago

dynamobi-build commented 12 years ago

[reporter="jvs", created="Wed, 20 Feb 2008 01:18:24 -0500 (GMT-05:00)"] I am putting in some special casing in NativeTrace so that the loggers we create for Segments and ExecStreams don't leak (when disabled) due to their per-object logger names.

New leaks could happen if additional tracing is added to Fennel with per-object logger names, so this issue is here to remind me that a more general scheme is needed. And maybe at the same time come up with a way to avoid leaks even when these are enabled?

Maybe the per-object naming should go as a prefix on each message (rather than as part of the logger name)?

dynamobi-build commented 12 years ago

[author="jvs", created="Wed, 20 Feb 2008 16:23:10 -0500 (GMT-05:00)"] See eigenchange 10585 for details of special-casing.