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

SPARQL 1.2 support #182

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

rubensworks
Copy link
Collaborator

This WIP PR adds support for the new reifier and triple terms syntax in SPARQL 1.2.

While it is too early to merge this PR, I've started this draft PR just to raise awareness that this is being worked on, to avoid unnecessary duplicated efforts.

Comment on lines +50 to +112
"subject": {
"termType": "BlankNode",
"value": "g_0"
},
"predicate": {
"termType": "Variable",
"value": "p"
},
"object": {
"termType": "Variable",
"value": "o"
}
},
{
"termType": "Quad",
"value": "",
"subject": {
"termType": "NamedNode",
"value": "http://example.com/ns#iri"
},
"predicate": {
"termType": "NamedNode",
"value": "http://www.w3.org/1999/02/22-rdf-syntax-ns#reifies"
},
"object": {
"termType": "Quad",
"value": "",
"subject": {
"termType": "BlankNode",
"value": "g_0"
},
"predicate": {
"termType": "Variable",
"value": "p"
},
"object": {
"termType": "Variable",
"value": "o"
},
"graph": {
"termType": "DefaultGraph",
"value": ""
}
},
"graph": {
"termType": "DefaultGraph",
"value": ""
}
},
{
"subject": {
"termType": "NamedNode",
"value": "http://example.com/ns#iri"
},
"predicate": {
"termType": "NamedNode",
"value": "http://example.com/ns#r"
},
"object": {
"termType": "Variable",
"value": "Z"
}
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This first reificationBlock (g_0) has 2 triple entries. This is in line with previous test results.
The second block (g1) has only one entry, missing a triple in the form of g_1 rdf:reifies quad .
Is this intended? If so, can you explain why?

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.

2 participants