[Bug 51919] Random ConcurrentModificationException or NoSuchElementException in CookieManager#removeMatchingCookies when using Concurrent Download

Hello Milamber, Sebb, All,
Regarding 51919 <https://issues.apache.org/bugzilla/show_bug.cgi?id=51919>,
I wonder if there is not an issue in JMeterVariables access introduced by
concurrent download.
Initially I think JMeterVariables were not supposed to be shared so object
not thread safe, today they are due to Conc download.
Maybe JMeterVariables#variables should be replaced by a ConcurrentHashMap.
If so CONTEXT_VARIABLES_LOCK should be removed in my patch.
What do you think?

Regards
Philippe


On Sat, Oct 1, 2011 at 9:34 AM, <[email protected]> wrote:

> https://issues.apache.org/bugzilla/show_bug.cgi?id=51919
>
> --- Comment #2 from Milamber <[email protected]> 2011-10-01 07:34:04 UTC
> ---
> A better way is to synchronize only the code section referred to the
> variables
> from JMeterContext
> (in add() and removeMatchingCookies() methods, I thinks)
>
> --
> Configure bugmail:
> https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.
> You reported the bug.
>



-- 
Cordialement.
Philippe Mouawad.
Ubik-Ingénierie