You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by rv...@apache.org on 2012/09/01 02:29:46 UTC

svn commit: r1379680 - /jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/core/DynamicDatasets.java

Author: rvesse
Date: Sat Sep  1 00:29:45 2012
New Revision: 1379680

URL: http://svn.apache.org/viewvc?rev=1379680&view=rev
Log:
Adds a proposed fix (currently commented out) to DynamicDatasets for the case where the provided description is empty, not enabled because I'm not sure what the interaction with defaultUnionGraph should be here

Modified:
    jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/core/DynamicDatasets.java

Modified: jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/core/DynamicDatasets.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/core/DynamicDatasets.java?rev=1379680&r1=1379679&r2=1379680&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/core/DynamicDatasets.java (original)
+++ jena/trunk/jena-arq/src/main/java/com/hp/hpl/jena/sparql/core/DynamicDatasets.java Sat Sep  1 00:29:45 2012
@@ -53,6 +53,10 @@ public class DynamicDatasets
     {
         if ( description == null )
             return dsg ;    
+//    	//An empty description means leave the dataset as-is
+//    	if (description.getDefaultGraphURIs().size() == 0 && description.getNamedGraphURIs().size() == 0) {
+//    		return dsg;
+//    	}
         Set<Node> defaultGraphs = NodeUtils.convertToNodes(description.getDefaultGraphURIs()) ; 
         Set<Node> namedGraphs = NodeUtils.convertToNodes(description.getNamedGraphURIs()) ;
         return dynamicDataset(defaultGraphs, namedGraphs, dsg, defaultUnionGraph) ;