Hi, On Thu, Jan 30, 2014 at 10:49 AM, Michael Dürig <[email protected]> wrote: > Being burnt by OAK-1370... but his might also lead to concurrent access to > the same session from multiple threads. We should probably also synchronise > the session related part of the NPM.
I'd simply synchronize the entire NPM. Doing so shouldn't cost too much in the normal case where just a single thread is accessing a session, but should avoid problems in the cases where there is concurrent access. In the case of observation this could even happen with a perfectly valid client that doesn't try to use the session concurrently from multiple threads, as observation thread will in any case need to access the NPM. BR, Jukka Zitting
