Jukka,

On 28.8.13 3:33 , [email protected] wrote:
Author: jukka
Date: Wed Aug 28 01:33:21 2013
New Revision: 1518048

URL: http://svn.apache.org/r1518048
Log:
OAK-150: Basic JCR LockManager support

...


      @Override
+    public boolean isRefresh() {
+        return true;
+    }
+


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.

Michael

Reply via email to