[
https://issues.apache.org/jira/browse/JENA-1430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16270617#comment-16270617
]
ASF GitHub Bot commented on JENA-1430:
--------------------------------------
Github user afs commented on a diff in the pull request:
https://github.com/apache/jena/pull/314#discussion_r153751942
--- Diff:
jena-arq/src/main/java/org/apache/jena/sparql/core/assembler/InMemDatasetAssembler.java
---
@@ -20,54 +20,45 @@
import static org.apache.jena.assembler.JA.data;
import static org.apache.jena.query.DatasetFactory.createTxnMem;
-import static org.apache.jena.query.ReadWrite.WRITE;
import static org.apache.jena.riot.RDFDataMgr.read;
import static
org.apache.jena.sparql.core.assembler.AssemblerUtils.setContext;
-import static
org.apache.jena.sparql.core.assembler.DatasetAssemblerVocab.pGraphName;
-import static
org.apache.jena.sparql.core.assembler.DatasetAssemblerVocab.pNamedGraph;
-import static
org.apache.jena.sparql.util.graph.GraphUtils.getAsStringValue;
-import static
org.apache.jena.sparql.util.graph.GraphUtils.multiValueResource;
+import static
org.apache.jena.sparql.core.assembler.DatasetAssemblerVocab.*;
+import static org.apache.jena.sparql.util.graph.GraphUtils.*;
import org.apache.jena.assembler.Assembler;
import org.apache.jena.assembler.Mode;
-import org.apache.jena.assembler.assemblers.AssemblerBase;
import org.apache.jena.query.Dataset;
import org.apache.jena.rdf.model.Resource;
+import org.apache.jena.system.Txn;
/**
* An {@link Assembler} that creates in-memory {@link Dataset}s.
*/
-public class InMemDatasetAssembler extends AssemblerBase implements
Assembler {
+public class InMemDatasetAssembler extends DatasetAssembler {
public static Resource getType() {
- return DatasetAssemblerVocab.tDatasetTxnMem ;
+ return DatasetAssemblerVocab.tMemoryDataset ;
}
@Override
public Dataset open(final Assembler assembler, final Resource root,
final Mode mode) {
- checkType(root, DatasetAssemblerVocab.tDatasetTxnMem);
--- End diff --
We ought to also check for use of the RDFDataset vocabulary on a
MemoryDataset. e.g. `ja:defaultGraph` and then `ja:namedGraph` pointing to
`ja:graph` and a `ja:MemoryModel`.
The reverse tests ought to be in general assembler.
> Quad loading for in-memory assemblers
> -------------------------------------
>
> Key: JENA-1430
> URL: https://issues.apache.org/jira/browse/JENA-1430
> Project: Apache Jena
> Issue Type: Bug
> Components: ARQ
> Reporter: A. Soroka
> Assignee: A. Soroka
>
> In-memory dataset Assemblers should support loading quad files.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
