Message8793

My concerns are:

1. We would need to do for all Python threads upon entry into Python context (possibly calling back from Java). The natural point to do this in this method in org.python.core.Py:

public static final ThreadState getThreadState(PySystemState newSystemState)

Because of the way ThreadState is passed through calls, this will not be called all the time, but it's certainly a hot path.

2. Such a hard reference from a thread to a class loader will prevent the class loader from being GCed (see this relevant email thread http://sourceforge.net/p/jython/mailman/message/32486365/), unless there's some sort of cleanup. See for example this highly relevant discussion on the Tomcat wiki: http://wiki.apache.org/tomcat/MemoryLeakProtection#cclThreadSpawnedByWebApp