You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by ju...@apache.org on 2013/09/18 20:22:03 UTC

svn commit: r1524517 - /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/ItemImpl.java

Author: jukka
Date: Wed Sep 18 18:22:03 2013
New Revision: 1524517

URL: http://svn.apache.org/r1524517
Log:
OAK-141: issues related to Node.refresh() as opposed to Session.refresh()

Only log a warning for Item.refresh(false) calls as there the current implementation can cause unsaved data to be lost.
For Item.refresh(true) there is no big problem in using Session.refresh(true) instead, as the session could in any case auto-refresh at that point.

Modified:
    jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/ItemImpl.java

Modified: jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/ItemImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/ItemImpl.java?rev=1524517&r1=1524516&r2=1524517&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/ItemImpl.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/session/ItemImpl.java Wed Sep 18 18:22:03 2013
@@ -289,7 +289,9 @@ abstract class ItemImpl<T extends ItemDe
      */
     @Override
     public void refresh(final boolean keepChanges) throws RepositoryException {
-        log.warn("Item#refresh invokes Session#refresh!");
+        if (!keepChanges) {
+            log.warn("Item#refresh invokes Session#refresh!");
+        }
         perform(new SessionOperation<Void>() {
             @Override
             public Void perform() throws InvalidItemStateException {