-
Notifications
You must be signed in to change notification settings - Fork 20
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
Blank node skolemization #6
Comments
On the one hand, this seems to be a problem with the HDT library not correctly converting blank nodes to the corresponding Jena representation. On the other hand, the TPF spec clearly says that components must not be blank nodes, so we should indeed skolemize them in any case, like the JavaScript implementation does. |
This might help, although reprocessing all nodes doesn't seem very performant. |
We should be able to do the same as in the JavaScript code by just changing this function (probably on the base class level even). |
Right, that would be fairly easy, but it would be datasource specific though... Another option would be to create a decorator for https://jena.apache.org/documentation/javadoc/arq/org/apache/jena/riot/WriterGraphRIOT.html |
Well, in the JavaScript version, it's implemented on the base class, so not source-specific. I still think this is possible. The additional complexity here is that |
So we'll have to improve the java HDT code no matter what, which gives us the opportunity to move to Jena 3 |
merging in latest changes from origin
Hi all, I just stumbled over this and it still seems to be an issue. I'm working on some other things in Server.java (including support for quad formats) so if you can give me any hints on how to fix this issue, I can give it a try. |
As per comunica/comunica#375 the spec now says that data triples MUST NOT contain blank nodes and that the RECOMMENDED way of removing them is skolemization. |
Is there such a thing as a conformance test suite for TPF servers? |
Not yet unfortunately, but that would indeed be very nice to have. |
Sounds like fun. You can assign it to me, I'll try it as friday afternoon thing |
HDT files with blank nodes generate invalid turtle because of blank nodes. Can we create a skolemized Model somehow?
The text was updated successfully, but these errors were encountered: