Add execConstructDataset(Dataset)

Project: http://git-wip-us.apache.org/repos/asf/jena/repo
Commit: http://git-wip-us.apache.org/repos/asf/jena/commit/cfb7fda2
Tree: http://git-wip-us.apache.org/repos/asf/jena/tree/cfb7fda2
Diff: http://git-wip-us.apache.org/repos/asf/jena/diff/cfb7fda2

Branch: refs/heads/master
Commit: cfb7fda26273c129c2e5bf6650169a89c06c67ac
Parents: 060248b
Author: Andy Seaborne <[email protected]>
Authored: Thu Aug 20 11:30:47 2015 +0100
Committer: Andy Seaborne <[email protected]>
Committed: Thu Aug 20 11:30:47 2015 +0100

----------------------------------------------------------------------
 .../jena/sparql/engine/QueryExecutionBase.java  | 24 ++++++++------------
 1 file changed, 10 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/cfb7fda2/jena-arq/src/main/java/org/apache/jena/sparql/engine/QueryExecutionBase.java
----------------------------------------------------------------------
diff --git 
a/jena-arq/src/main/java/org/apache/jena/sparql/engine/QueryExecutionBase.java 
b/jena-arq/src/main/java/org/apache/jena/sparql/engine/QueryExecutionBase.java
index 9a98b71..c04675b 100644
--- 
a/jena-arq/src/main/java/org/apache/jena/sparql/engine/QueryExecutionBase.java
+++ 
b/jena-arq/src/main/java/org/apache/jena/sparql/engine/QueryExecutionBase.java
@@ -47,7 +47,6 @@ import org.apache.jena.riot.system.IRIResolver;
 import org.apache.jena.shared.PrefixMapping;
 import org.apache.jena.sparql.ARQConstants;
 import org.apache.jena.sparql.core.DatasetGraph;
-import org.apache.jena.sparql.core.DatasetGraphFactory;
 import org.apache.jena.sparql.core.Quad;
 import org.apache.jena.sparql.core.describe.DescribeHandler;
 import org.apache.jena.sparql.core.describe.DescribeHandlerRegistry;
@@ -279,27 +278,24 @@ public class QueryExecutionBase implements QueryExecution
     
     @Override
     public Dataset execConstructDataset(){
-       
-       DatasetGraph graph = DatasetGraphFactory.createMem();
-       
-        checkNotClosed() ;
-        try
-        {
-               execConstructQuads().forEachRemaining(graph::add);
-        }
-        finally
-        {
+        return execConstructDataset(DatasetFactory.createMem()) ;
+    }
+
+    @Override
+    public Dataset execConstructDataset(Dataset dataset){
+        DatasetGraph dsg = dataset.asDatasetGraph() ; 
+        try {
+            execConstructQuads().forEachRemaining(dsg::add);
+        } finally {
             this.close();
         }
-        return DatasetFactory.create(graph);
-
+        return dataset ; 
     }
 
     @Override
     public Model execDescribe()
     { return execDescribe(GraphFactory.makeJenaDefaultModel()) ; }
 
-
     @Override
     public Model execDescribe(Model model)
     {

Reply via email to