solid-spec icon indicating copy to clipboard operation
solid-spec copied to clipboard

Drop SPARQL query functionality on GET

Open RubenVerborgh opened this issue 6 years ago • 6 comments

This should likely not be in the core spec (especially since it is unclear what subset should be supported). Can be in an extension though.

RubenVerborgh avatar May 07 '19 11:05 RubenVerborgh

@RubenVerborgh Just so I understand the implications of this: Would this mean moving the parts on SPARQL and/or SPARQL UPDATE out of LDP?

Liked this in general because I think it's always good to simplify specifications where possible, but realized I might not understand the implications of this.

megoth avatar May 12 '19 17:05 megoth

Also, if yes, that it means moving SPARQL out of LDP, we should thread very carefully. This is functionality that both rdflib.js and LDflex is dependent on, ain't it?

megoth avatar May 12 '19 17:05 megoth

This is a proposal for moving SPARQL query functionality out of the spec. Using SPARQL UPDATE as body of a PATCH request can stay.

LDflex does not depend on server-side SPARQL queries; I don’t think any lib/app does, since NSS doesn’t support it.

RubenVerborgh avatar May 12 '19 17:05 RubenVerborgh

Ok, as long as SPARQL UPDATEs as part of body stays, I don't have any objections ^_^

megoth avatar May 12 '19 19:05 megoth

And thanks for clarifying ^_^

megoth avatar May 12 '19 19:05 megoth

I don't see the point of removing this.

it is unclear what subset should be supported

So to fix that, let's document it better! :)

I'm not very familiar with SPARQL, but to me it seemed quite simple you can do any SPARQL query on an RDF source, as long as it's valid SPARQL? Can you elaborate on what is unclear?

michielbdejong avatar May 14 '19 09:05 michielbdejong