diff options
Diffstat (limited to 'jdisc_core')
| -rw-r--r-- | jdisc_core/src/main/java/com/yahoo/jdisc/core/ActiveContainer.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/jdisc_core/src/main/java/com/yahoo/jdisc/core/ActiveContainer.java b/jdisc_core/src/main/java/com/yahoo/jdisc/core/ActiveContainer.java index 692c669780f..51a84a4a6c6 100644 --- a/jdisc_core/src/main/java/com/yahoo/jdisc/core/ActiveContainer.java +++ b/jdisc_core/src/main/java/com/yahoo/jdisc/core/ActiveContainer.java @@ -67,12 +67,13 @@ public class ActiveContainer extends AbstractResource implements CurrentContaine termination.run(); } + // TODO Get rid of finalizer and use PhantomReference or Java 9 Cleaner instead @Override protected void finalize() throws Throwable { try { int retainCount = retainCount(); if (retainCount > 0) { - log.warning(this + ".destroy() invoked from finalize() not through ApplicationLoader. " + + log.severe("Destructing " + this + " through finalizer since reference count never reached zero. " + "This is an indication of either a resource leak or invalid use of reference counting. " + "Retained references as this moment: " + retainCount); destroy(); |