commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (COMMONSRDF-55) Stream of Jena quads use wrong IRI for default graph
Date Fri, 03 Feb 2017 11:32:51 GMT


ASF GitHub Bot commented on COMMONSRDF-55:

Github user afs commented on the issue:
    I haven't looked at the code in enough detail but I was assuming there would be one place
with two functions: Quad->JenaQuad and JenaQuad->Quad. This would the strategy (policy)
for all of the adapter. That is what I mean by the boundary.
    "urn:x-arq:DefaultGraph" only exists in Jena. Quad has `Optional.empty()`
    `Optional.empty()` => `urn:x-arq:DefaultGraph`
    Jena `Quad.isDefaultGraph` => `Optional.empty()`
    `Quad.equals` works, which is very important.
    or maybe I've been doing too much policy abstraction recently.

> Stream of Jena quads use wrong IRI for default graph
> ----------------------------------------------------
>                 Key: COMMONSRDF-55
>                 URL:
>             Project: Apache Commons RDF
>          Issue Type: Bug
>          Components: jena
>    Affects Versions: 0.3.0
>            Reporter: Stian Soiland-Reyes
>            Assignee: Stian Soiland-Reyes
>             Fix For: 1.0.0
> See
> {code}
> org.apache.commons.rdf.jena.DatasetJenaTest
> streamLanguageTagsCaseInsensitive(org.apache.commons.rdf.jena.DatasetJenaTest)  Time
elapsed: 0.012 sec  <<< FAILURE!
> java.lang.AssertionError: expected:<<> <>
"Hello"@EN-GB <urn:x-arq:DefaultGraph>.> but was:<<>
<> "Hello"@en-GB .>
> {code}
> Jena uses the IRI `<urn:x-arq:DefaultGraph>` internally to represent the default
graph within datasets - we need to recognize that on the way out of a ``
and possibly in the `asQuad(JenaQuad)` converter and replace it with `Optional.empty()` so
the default graph appears the same across implementations.
> The `AbstractDatasetTest`  should be augmented to do more tests on the default graph,
including `.stream()`, `.iterate()`, `.contains()` and `.remove()`1.

This message was sent by Atlassian JIRA

View raw message