[weld-commits] [weld/core] 424245: WELD-762 Test Case showing ThreadLocal leak in Wel...

[noreply at github.com]

Branch: refs/heads/master
Home:   https://github.com/weld/core

Commit: 42424567c207c87a2d835b75089db2530285349d
    https://github.com/weld/core/commit/42424567c207c87a2d835b75089db2530285349d
Author: Aslak Knutsen <aslak at redhat.com>
Date:   2010-11-30 (Tue, 30 Nov 2010)

Changed paths:
  A tests-arquillian/src/test/java/org/jboss/weld/tests/threadlocal/Foo.java
  A tests-arquillian/src/test/java/org/jboss/weld/tests/threadlocal/ThreadLocalTestCase.java

Log Message:
-----------
WELD-762 Test Case showing ThreadLocal leak in Weld Core after call to container.stopContainer(). This is not the same ThreadLocal leak as reported in Issue, could not reproduce.


Commit: 23c381ecb24d5ab5da981679164d0c4eaedac830
    https://github.com/weld/core/commit/23c381ecb24d5ab5da981679164d0c4eaedac830
Author: Pete Muir <pmuir at bleepbleep.org.uk>
Date:   2010-11-30 (Tue, 30 Nov 2010)

Changed paths:
  M impl/src/main/java/org/jboss/weld/context/AbstractManagedContext.java
  M impl/src/main/java/org/jboss/weld/context/unbound/RequestContextImpl.java
  A tests-arquillian/src/test/java/org/jboss/weld/tests/threadlocal/Bar.java
  A tests-arquillian/src/test/java/org/jboss/weld/tests/threadlocal/Baz.java
  M tests-arquillian/src/test/java/org/jboss/weld/tests/threadlocal/ThreadLocalTestCase.java

Log Message:
-----------
WELD-762

* Looks like leak was just due to the way test
  cleaned up contexts
* Alterted contexts use of ThreadLocal's to prevent
  possibility of leak if contexts don't get cleared
  up