You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2014/08/06 18:57:36 UTC

svn commit: r1616284 - /jena/Experimental/jena-fuseki2/src/main/java/org/apache/jena/fuseki/mgt/ActionDatasets.java

Author: andy
Date: Wed Aug  6 16:57:36 2014
New Revision: 1616284

URL: http://svn.apache.org/r1616284
Log:
Notes

Modified:
    jena/Experimental/jena-fuseki2/src/main/java/org/apache/jena/fuseki/mgt/ActionDatasets.java

Modified: jena/Experimental/jena-fuseki2/src/main/java/org/apache/jena/fuseki/mgt/ActionDatasets.java
URL: http://svn.apache.org/viewvc/jena/Experimental/jena-fuseki2/src/main/java/org/apache/jena/fuseki/mgt/ActionDatasets.java?rev=1616284&r1=1616283&r2=1616284&view=diff
==============================================================================
--- jena/Experimental/jena-fuseki2/src/main/java/org/apache/jena/fuseki/mgt/ActionDatasets.java (original)
+++ jena/Experimental/jena-fuseki2/src/main/java/org/apache/jena/fuseki/mgt/ActionDatasets.java Wed Aug  6 16:57:36 2014
@@ -275,23 +275,27 @@ public class ActionDatasets extends Acti
 
     @Override
     protected void execDeleteItem(HttpAction action) {
+//      if ( isContainerAction(action) ) {
+//      ServletOps.errorBadRequest("DELETE only applies to a specific dataset.") ;
+//      return ;
+//  }
+  
         // Does not exist?
         String name = action.getDatasetName() ;
         if ( name == null )
             name = "" ;
         action.log.info(format("[%d] DELETE ds=%s", action.id, name)) ;
 
-        if ( isContainerAction(action) ) {
-            ServletOps.errorBadRequest("DELETE only applies to a specific dataset.") ;
-            return ;
-        }
-        
         if ( ! DataAccessPointRegistry.get().isRegistered(name) )
             ServletOps.errorNotFound("No such dataset registered: "+name);
 
         systemDSG.begin(ReadWrite.WRITE) ;
-        boolean committed =false ;
+        boolean committed = false ;
         try {
+            // Here, go offline.
+            // Need to reference count operations when they drop to zero
+            // or a timer goes off, we delete the dataset.
+            
             DataAccessPoint ref = DataAccessPointRegistry.get().get(name) ;
             // Redo check inside transaction.
             if ( ref == null )