On 11.9.13 3:38 , Jukka Zitting wrote:
Hi,
On Wed, Sep 11, 2013 at 5:19 AM, Michael Dürig <[email protected]> wrote:
What is the reason for returning true here? Only operations that refresh the
session should return true here. If returning true is necessary here, you
might be exploiting a side effect and we should try to come up with a more
explicit solution.
The rationale behind this is that the lock operations (since they are
inherently cross-session ones) always refresh the session to the
latest state (see LockOperation.perform). It's not fully correct from
a strict locking semantics point of view, but achieves the kind of
"fuzzy locking" functionality I was aiming at.
Makes sense, thanks for clarifying. I initially didn't spot the
overloaded perform(NodeDelegate), which the initial perform delegates to.
Michael
BR,
Jukka
