Page 1 of 1

OpenDoPEHandler is not thread-safe

PostPosted: Fri May 20, 2011 2:24 am
by tinne
One cannot run OpenDoPEHandler twice in the same jvm at the same time, as the static field shallowTraversor might be accessed in the wrong way by two parallel runs of preprocess.

The easiest way to cure this might be to make everything non-static. The most portable way might be to add some static helper methods to aid porting.

As an instant cure, I suggest adding a warning and/or synchronizing preprocess.

Re: OpenDoPEHandler is not thread-safe

PostPosted: Thu Aug 11, 2011 12:18 am
by olap
Hi!

It looks like current implementation is not thread-safe :(
Is there any chance to fix this in the nearest future ?

Thanks,
Oleg

Re: OpenDoPEHandler is not thread-safe

PostPosted: Thu Aug 11, 2011 1:39 am
by jason
I had no immediate plans to work on it.

That said, I don't think its a big task. I'll have a look at in tomorrow or more likely Friday. Alternatively, if you'd like to make everything non-static yourself, I'd be happy to accept the patch.

Re: OpenDoPEHandler is not thread-safe

PostPosted: Fri Aug 12, 2011 3:09 pm
by jason