forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tim Williams" <william...@gmail.com>
Subject Re: Concurrency testing of the locationmap throws errors
Date Mon, 30 Jun 2008 13:14:32 GMT
On Mon, Jun 30, 2008 at 9:04 AM, Thorsten Scherler
<thorsten.scherler.ext@juntadeandalucia.es> wrote:
> On Mon, 2008-06-30 at 08:34 -0400, Tim Williams wrote:
>> On Mon, Jun 30, 2008 at 8:12 AM, Thorsten Scherler
> ...
>>
>> I reckon in that case, it's either the caching or its deeper in the
>> actual LM/nodes.  You could try turning off caching completely and see
>> if it helps.
>
> Turning caching off does not change the situation.
>
>> Or, you could change the implementation from a HashMap
>> to a ConcurrentHashMap and see if that helps.
>
> I changed
> public synchronized Object getAttribute(...)
> which seems to fix the problem.
>
> :)

I'm not sure that having getAttribute synchronized is good long term.
Have you noticed a performance problems now?  Maybe create an issue to
review this and figure out what the real culprit is in there?

--tim

Mime
View raw message