2015-01-08 14:50 GMT+01:00 Mark Thomas <ma...@apache.org>: > > There's a big deadlock risk if the recursion if incomplete and error > > processing are inside the sync. > > I came across some of those while doing the refactoring. I think - as of > r1650289 - they are addressed but it is possible I missed something. If > you have a particular sequence in mind then I'd be happy to take another > look. > > If that new code is also used in upgrade mode, then abusing the drawboard in localhost is a good way to find deadlocks and corruption. But I don't see why these shouldn't be out of the sync block even if it doesn't cause a deadlock.
Rémy