commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From stain <...@git.apache.org>
Subject [GitHub] commons-rdf issue #32: COMMONSRDF-55: Handle Jena's urn:x-arq:DefaultGraph a...
Date Wed, 01 Feb 2017 15:06:58 GMT
Github user stain commented on the issue:

    https://github.com/apache/commons-rdf/pull/32
  
    Any comments from perhaps @afs or @ajs6f? I'm not sure exactly at what boundary we should
replace `<urn:x-arq:DefaultGraph>` and friends with `Optional.empty()` -- in the solution
of this PR it happens for any Quad created with `JenaRDF` - but of course that would not happen
in other `RDF` instances. This suggestion would also convert any "foreign" `IRI` instances
of `<urn:x-arq:DefaultGraph>` when given as a graph name.
    
    I'll admit there's potential for information loss if the alternate  `<urn:x-arq:DefaultGraphNode>`
is used as a string IRI or Node when making a Commons RDF Quad  - this PR will adapt it into
`<urn:x-arq:DefaultGraph> when later making the Jena `Quad` instance from the field
of Optional.empty(), however Jena's `DatasetGraph` also does that conversion.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message