CODEWITHSUNDEEP
javagarbage-collectionclassloader
michael nesterenko
michael nesterenko

Reputation: 14449

ThreadLocal garbage collection

From javadoc

Each thread holds an implicit reference to its copy of a thread-local variable as long as the thread is alive and the ThreadLocal instance is accessible; after a thread goes away, all of its copies of thread-local instances are subject to garbage collection (unless other references to these copies exist).

from that it seems that objects referenced by a ThreadLocal variable are garbage collected only when thread dies. But what if ThreadLocal variable a is no more referenced and is subject for garbage collection? Will object references only by variable a be subject to garbage collection if thread that holds a is still alive?

for example there is following class with ThreadLocal variable:

public class Test {
    private static final ThreadLocal a = ...; // references object b
}

This class references some object and this object has no other references to it. Then during context undeploy application classloader becomes a subject for garbage collection, but thread is from a thread pool so it does not die. Will object b be subject for garbage collection?

Upvotes: 24

Views: 9653

Answers (6)

Stephen C
Stephen C

Reputation: 719199

TL;DR : You cannot count on the value of a ThreadLocal being garbage collected when the ThreadLocal object is no longer referenced. You have to call ThreadLocal.remove or cause the thread to terminate

(Thanks to @Lii)

Detailed answer:

from that it seems that objects referenced by a ThreadLocal variable are garbage collected only when thread dies.

That is an over-simplification. What it actually says is two things:

  • The value of the variable won't be garbage collected while the thread is alive (hasn't terminated), AND the ThreadLocal object is strongly reachable.

  • The value will be subject to normal garbage collection rules when the thread terminates.

There is an important third case where the thread is still live but the ThreadLocal is no longer strongly reachable. That is not covered by the javadoc. Thus, the GC behavior in that case is unspecified, and could potentially be different across different Java implementations.

In fact, for OpenJDK Java 6 through OpenJDK Java 17 (and other implementations derived from those code-bases) the actual behavior is rather complicated. The values of a thread's thread-locals are held in a ThreadLocalMap object. The comments say this:

ThreadLocalMap is a customized hash map suitable only for maintaining thread local values. [...] To help deal with very large and long-lived usages, the hash table entries use WeakReferences for keys. However, since reference queues are not used, stale entries are guaranteed to be removed only when the table starts running out of space.

If you look at the code, stale map entries (with broken WeakReferences) may also be removed in other circumstances. If stale entry is encountered in a get, set, insert or remove operation on the map, the corresponding value is nulled. In some cases, the code does a partial scan heuristic, but the only situation where we can guarantee that all stale map entries are removed is when the hash table is resized (grows).

So ...

Then during context undeploy application classloader becomes a subject for garbage collection, but thread is from a thread pool so it does not die. Will object b be subject for garbage collection?

The best we can say is that it may be ... depending on how the application manages other thread locals the thread in question.

So yes, stale thread-local map entries could be a storage leak if you redeploy a webapp, unless the web container destroys and recreates all of the request threads in the thread pool. (You would hope that a web container would / could do that, but AFAIK it is not specified.)

The other alternative is to have your webapp's servlets always clean up after themselves by calling ThreadLocal.remove on each one on completion (successful or otherwise) of each request.

Upvotes: 11

Evgeniy Dorofeev
Evgeniy Dorofeev

Reputation: 136062

ThreadLocal variables are hold in Thread

ThreadLocal.ThreadLocalMap threadLocals;

which is initialized lazily on first ThreadLocal.set/get invocation in the current thread and holds reference to the map until Thread is alive. However ThreadLocalMap uses WeakReferences for keys so its entries may be removed when ThreadLocal is referenced from nowhere else. See ThreadLocal.ThreadLocalMap javadoc for details

Upvotes: 8

user1944408
user1944408

Reputation: 569

Object b will not be subject for garbage collection if it somehow refers to your Test class. It can happen without your intention. For example if you have a code like this:

public class Test {
    private static final ThreadLocal<Set<Integer>> a =
     new ThreadLocal<Set<Integer>>(){
            @Override public Set<Integer> initialValue(){
                return new HashSet<Integer>(){{add(5);}};
            }
    };
}

The double brace initialization {{add(5);}} will create an anonymous class which refers to your Test class so this object will never be garbage collected even if you don't have reference to your Test class anymore. If that Test class is used in a web app then it will refer to its class loader which will prevent all other classes to be GCed.

Moreover, if your b object is a simple object it will not be immediately subject for GC. Only when ThreadLocal.ThreadLocalMap in Thread class is resized you will have your object b subject for GC.

However I created a solution for this problem so when you redeploy your web app you will never have class loader leaks.

Upvotes: 0

Frank Pavageau
Frank Pavageau

Reputation: 11715

If the ThreadLocal itself is collected because it's not accessible anymore (there's an "and" in the quote), then all its content can eventually be collected, depending on whether it's also referenced somewhere else and other ThreadLocal manipulations happen on the same thread, triggering the removal of stale entries (see for example the replaceStaleEntry or expungeStaleEntry methods in ThreadLocalMap). The ThreadLocal is not (strongly) referenced by the threads, it references the threads: think of ThreadLocal<T> as a WeakHashMap<Thread, T>.

In your example, if the classloader is collected, it will unload the Test class as well (unless you have a memory leak), and the ThreadLocal a will be collected.

Upvotes: 7

gma
gma

Reputation: 2563

It depends, it will not be garbage collected if your are referencing it as static or by singleton and your class is not unloaded, that is why in application server environment and with ThreadLocal values, you have to use some listener or request filter the be sure that you are dereferencing all thread local variables at the end of the request processing. Or either use some Request scope functionality of your framework.

You can look here for some other explanations.

EDIT: In the context of a thread pool as asked, of course if the Thread is garbaged thread locals are.

Upvotes: 0

anish
anish

Reputation: 7412

ThreadLocal contains a reference to a WeakHashMap that holds key-value pairs

enter image description here

Upvotes: 3

Related Questions