Alan D. Cabrera wrote:
I notice that the Correlator class uses a WeakHashMap to store the FutureResults. However, the values in the map have strong references to thier keys. This means that the elements in the map never get removed and you may as well use a HashMap. Did I miss something?
Doesn't sound like it - WeakHashMap is for weak keys and this sounds like it needs weak values. There may be a version around that does this - if not, there's something that might work in the old lock code:
http://cvs.apache.org/viewcvs.cgi/incubator-geronimo/modules/core/src/java/org/apache/geronimo/lock/LockDomain.java?rev=1.3&view=auto
-- Jeremy
