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 an...@apache.org on 2012/05/09 12:41:27 UTC

svn commit: r1336067 - in /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak: core/ContentSessionImpl.java kernel/CoreValueFactoryImpl.java

Author: angela
Date: Wed May  9 10:41:27 2012
New Revision: 1336067

URL: http://svn.apache.org/viewvc?rev=1336067&view=rev
Log:
TODOs and minor improvement

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/ContentSessionImpl.java
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/kernel/CoreValueFactoryImpl.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/ContentSessionImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/ContentSessionImpl.java?rev=1336067&r1=1336066&r2=1336067&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/ContentSessionImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/core/ContentSessionImpl.java Wed May  9 10:41:27 2012
@@ -37,8 +37,7 @@ import java.util.Set;
 class ContentSessionImpl implements ContentSession {
 
     /** Logger instance */
-    private static final Logger log =
-            LoggerFactory.getLogger(ContentSessionImpl.class);
+    private static final Logger log = LoggerFactory.getLogger(ContentSessionImpl.class);
 
     private final LoginContext loginContext;
     private final String workspaceName;

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/kernel/CoreValueFactoryImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/kernel/CoreValueFactoryImpl.java?rev=1336067&r1=1336066&r2=1336067&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/kernel/CoreValueFactoryImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/kernel/CoreValueFactoryImpl.java Wed May  9 10:41:27 2012
@@ -71,6 +71,7 @@ class CoreValueFactoryImpl implements Co
 
     @Override
     public CoreValue createValue(InputStream value) {
+        // TODO: mk.write throws MikrokernelException ... deal with this here.
         String binaryID = mk.write(value);
         return new CoreValueImpl(new BinaryValue(binaryID, mk));
     }