https://bz.apache.org/bugzilla/show_bug.cgi?id=62082

Philippe Mouawad <p.moua...@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om

--- Comment #2 from Philippe Mouawad <p.moua...@ubik-ingenierie.com> ---
(In reply to Vladimir Sitnikov from comment #1)
> It does look like the synchronization there is not needed.

Hi Vladimir,
Would you say that this answer is correct :

- https://stackoverflow.com/q/48648931/460802

IMO, it is.
If we remove the synchronization, my understanding is that:
- there is a risk if Groovy script uses global variables, there is no reason to
do that
- I am not sure about the risk on GroovyClassLoader. I don't think there is as
per what is described here:

http://docs.groovy-lang.org/latest/html/documentation/guide-integrating.html#groovyclassloader


Is my understanding correct ?
Thanks

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to