Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blanknode identifier only #64

Closed
wants to merge 13 commits into from
Closed

Blanknode identifier only #64

wants to merge 13 commits into from

Conversation

stain
Copy link
Contributor

@stain stain commented Feb 6, 2015

A variant proposal of #63

I putting this patch as a proposal for discussion, not giving my +1 yet.

tl;dr: BlankNode.identifier() returns a "globally unique identifier" - recommends UUID or Skolem IRI. Used for equality checks.

Added:

The identifier MUST be globally unique. It is RECOMMENDED for the identifier to be a UUID string. If the identifier is a IRI, it is SHOULD be a skolem IRI.

It is NOT RECOMMENDED for this identifier to form part of the RDFTerm.ntriplesString().

Javadoc: http://stain.github.io/commons-rdf/blanknode-identifier-only/

Same discussion as in #63 applies, except:

  • Implementations are more at freedom for what kind of identifier scheme they use
    • BUT: Must make those globally unique. "jena:graph5:b4" won't do. :)
  • Users of the API might be more likely to mess up by sending in b1 as the string to the RDFTermFactory.createBlankNode(string). Javadoc warning added.

@coveralls
Copy link

Coverage Status

Coverage decreased (-3.81%) to 80.11% when pulling b350c17 on stain:blanknode-identifier-only into 61f97fc on commons-rdf:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants