- From: Alexandre Passant <alexandre.passant@deri.org>
- Date: Tue, 9 Aug 2011 14:48:22 +0100
- To: SPARQL Working Group <public-rdf-dawg@w3.org>
- Message-Id: <895AA5C4-6F6A-4E41-A74E-1B88658167EB@deri.org>
Here's my (delayed) review of the federated query document - there are mostly editorial comments - minor ones except the first one that I think should be addressed before the publishing. This completes ACTION-501 Best, Alex. * TOC - Appendix C1 to be removed * 2 (intro): - You use "SPARQL protocol service" but in the rest of the document, "SPARQL endpoint". Is that the same thing ? If so, that should should be replaced, if not "protocol service" should be explained. Similarly, in 2.2 you use *service* ("Data in the default graph at service: http://people.example.org/sparql") and in 2.3 *endpoint* (Data in <http://people.example.org/sparql> endpoint). This is the only major concern I have with the current spec but I think that should be fixed. * 2.1 - "and we want to combine it with my local FOAF file". I'd rather use neutral pronouns. "one wants to combine with a local FOAF file". That's a general remark to avoid the use of "we" that sounds too tutorial-ish too me. * 2.2 - "Data in <http://people.example.org/sparql> endpoint". Endpoint URL should be in <code> * 2.4. - for all foaf:Persons. Put the "s" outside the <code>, or better, replace by "instances of foaf:Person" * 4 - "DOAP vocabulary" . Add an hyperlink of reference. - " void:sparqlEndpoint ?service " I think there's a . missing here -- Dr. Alexandre Passant, Social Software Unit Leader Digital Enterprise Research Institute, National University of Ireland, Galway
Received on Tuesday, 9 August 2011 13:48:51 UTC