Re: svn commit: r1166165 - in /jakarta/jmeter/trunk: src/protocol/http/org/apache/jmeter/protocol/http/control/CacheManager.java xdocs/changes.xml

On 7 September 2011 14:51, Milamber <[email protected]> wrote:
> On Wed, Sep 7, 2011 at 1:35 PM, sebb <[email protected]> wrote:
>
>> On 7 September 2011 14:20,  <[email protected]> wrote:
>> [snip]
>>
>> -    private transient ThreadLocal<Map<String, CacheEntry>> threadCache;
>> > +    private transient InheritableThreadLocal<Map<String, CacheEntry>>
>> threadCache;
>>
>> Neat fix!
>>
>
> Thanks.
> 2-3 modified lines, but some hours last sunday to find a elegant way to
> correcting this without success.

Definitely elegant.

> Finally, today I googling, and I discovered this thread:
> http://stackoverflow.com/questions/7259906/propagating-threadlocal-to-a-new-thread-fetched-from-a-executorservice
> (I didn't know this InheritableThreadLocal class before today)
>

Nor me, otherwise I might have thought of using it here.

We need to keep it in mind for any similar situations.

>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [email protected]
>> For additional commands, e-mail: [email protected]
>>
>>
>