[
https://issues.apache.org/jira/browse/COMMONSRDF-66?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16224964#comment-16224964
]
ASF GitHub Bot commented on COMMONSRDF-66:
------------------------------------------
Github user afs commented on a diff in the pull request:
https://github.com/apache/commons-rdf/pull/42#discussion_r147701840
--- Diff:
jena/src/main/java/org/apache/commons/rdf/jena/impl/JenaDatasetImpl.java ---
@@ -149,7 +149,7 @@ public long size() {
@Override
public String toString() {
final StringWriter sw = new StringWriter();
- RDFDataMgr.write(sw, graph, Lang.NT);
--- End diff --
It would good to rename graph (throughout the class) as `datasetGraph` or
some such. It's not a graph.
> JenaDatasetImpl.toString() throws RIOT exception
> ------------------------------------------------
>
> Key: COMMONSRDF-66
> URL: https://issues.apache.org/jira/browse/COMMONSRDF-66
> Project: Apache Commons RDF
> Issue Type: Bug
> Components: jena
> Affects Versions: 0.3.0
> Reporter: Christopher Johnson
>
> Occurs from this method
> [https://github.com/apache/commons-rdf/blob/master/jena/src/main/java/org/apache/commons/rdf/jena/impl/JenaDatasetImpl.java#L152]
> on instantiation:
> The exception message is "No dataset writer for N-Triples/utf-8".
> The source is RDFWriterRegistry that builds a registryDataset HashMap of 17
> +Dataset+ RDFFormats that does not include this serialization. The
> registryGraph HashMap has 25 RDFFormats that does include "N-Triples/utf-8".
> No exception is thrown if Lang.NQUADS is set in the toString() method.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
