This is an automated email from the ASF dual-hosted git repository.

rombert pushed a commit to branch master
in repository 
https://gitbox.apache.org/repos/asf/sling-org-apache-sling-mongodb.git

commit db60601327566ec0a98fe205d9373d97db9401c9
Author: Carsten Ziegeler <[email protected]>
AuthorDate: Thu Aug 2 06:21:32 2012 +0000

    New MongoDB resource provider
    
    git-svn-id: https://svn.apache.org/repos/asf/sling/trunk@1368357 
13f79535-47bb-0310-9956-ffa450edef68
---
 .../java/org/apache/sling/mongodb/impl/MongoDBResourceProvider.java | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/src/main/java/org/apache/sling/mongodb/impl/MongoDBResourceProvider.java 
b/src/main/java/org/apache/sling/mongodb/impl/MongoDBResourceProvider.java
index c35a8f1..eafb2b9 100644
--- a/src/main/java/org/apache/sling/mongodb/impl/MongoDBResourceProvider.java
+++ b/src/main/java/org/apache/sling/mongodb/impl/MongoDBResourceProvider.java
@@ -114,6 +114,11 @@ public class MongoDBResourceProvider implements 
ResourceProvider, ModifyingResou
     }
 
     /**
+     * TODO - we should handle delete different and not put all child 
resources into the
+     * deleted set.
+     * Instead when getting resources, the parents of the resource should be 
checked
+     * first.
+     * This minimizes concurrency issues.
      * @see 
org.apache.sling.api.resource.ModifyingResourceProvider#delete(org.apache.sling.api.resource.ResourceResolver,
 java.lang.String)
      */
     public void delete(final ResourceResolver resolver, final String path)
@@ -235,6 +240,7 @@ public class MongoDBResourceProvider implements 
ResourceProvider, ModifyingResou
     }
 
     /**
+     * TODO - we have to check for deleted and added resources
      * @see 
org.apache.sling.api.resource.ResourceProvider#listChildren(org.apache.sling.api.resource.Resource)
      */
     public Iterator<Resource> listChildren(final Resource parent) {

-- 
To stop receiving notification emails like this one, please contact
"[email protected]" <[email protected]>.

Reply via email to