-
Notifications
You must be signed in to change notification settings - Fork 2
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
Drop the requirement to support ill-typed literals with recognized datatype IRIs #60
Comments
The current text is a bit strange.
I don't think that the "MUST" can be meaningful if the literals are outside RDF-semantics. In RDF concepts, the text
Any system can issue warning for anything regardless of this text so it can be dropped or made advice text as encouragement to do that. For RDF Concepts , can we just say: which allows variation when there's justification. ("support" is stronger than "accept". "Accept" is about RDF terms (correct syntax). I would read "Support" is about acting, e.g. on the values c.f. D-entailment.) |
+1 Actually, I consider this bit of RDF Concepts to contradict RDF Semantics §7.2, which says:
and in fact some implementations already do :) This makes a strong case for replacing this MUST with a MAY in RDF-syntax, IMO. |
MAY is weak IMO. It would be nice to encourage the behavior of passing through syntactically correct data with "SHOULD accept ill-typed literals". |
This can be expressed as an advisory in the specification as a Note or within the Considerations section providing additional context for implementations to evaluate advantages and pitfalls. |
This was discussed during the rdf-star meeting on 31 October 2024. View the transcriptDrop the requirement to support ill-typed literals with recognized datatype IRIs 2pfps: I agree with what Andy says in the issue AndyS: it depends what "support" really means here AZ: I also want to ask what is ment by "support". If you have a system that does not recognize a datatype IRI <AndyS> RDF concepts -- "The list of datatypes supported by an implementation is determined by its recognized datatype IRIs." seems to be the nearest to defining "support". AZ: If you say this kind of graphs may not be supported, what about other kinds of inconsistencies. Should any such graph not be supported? pfps: one option would be to tweak the wording <pfps> One option is that implementations MUST accept input documents with ill-typed literals and SHOULD include the resultant triple in the RDF graph. gkellogg: it makes no sense to talk about an ill-type literal for non-recognised datatypes <pfps> That is - parsing MUST NOT stop at an ill-typed literal but the system MAY choose to not include the triple in the resultant graph. gkellogg: I think the idea is to be able to only retain well-typed literals <pfps> I would add that if an implementation drops the triple then it MUST produce a warning. gkellogg: it would be reasonable for RDF systems to not deal with ill-typed literals TallTed: the current text is "MUST accept", not "MUST support" ktk: how are different implementations dealing with this? AndyS: in SPARQL, there are cases when you need to assign a value, so it does not work with ill-typed literals but that a SPARQL process <pfps> agreed that it is difficult to require a warning james: We are very accepting (in our implem) and it has been very useful <AndyS> "SHOULD accept" -- MUST for warnings is a bit strange. We don't have a "warning" mechanism in the specs. james: but it's personal opinion Souri: when we find an ill-typed literal, we separate it AndyS: choosing the datatypes you choose to handle is something you do when you use the data TallTed: I'm concerned to hear that some implementations are not conformant <niklasl> +1 for evolution (with the caveat that I prefer opt-in "drop unrecognized" modes to avoid sending inexplicable data onward). Souri: if we have an xsd:integer with "abc" lexical form, we don't accept it, but if you have ex:mytype, we don't do anything <Zakim> pfps, you wanted to say that implementations that reject unrecognized datatypes are broken but ones that do not fully accept known ill-typed literals are not so bad james: we do 2 kind of things, one on the values to do efficient operation, and one that just take any literal transparently ktk: what do we do with this issue? We don't really have a conclusion <TallTed> "MUST accept" is current text Souri: in Oracle, we don't want to have, e.g., 31st February, so we reject it TallTed: not accepting data is bad but you can handle the ill-typed literals after they are loaded tl: I like the idea that there are several phases, 1st you parse and put in store, then other processes AndyS: I find the use cases of rejecting or not rejecting both reasonable <AndyS> My pref is change "MUST accept" to "SHOULD accept". All the described handling cases seem reasonable for their different cases. Souri: w do not reject entire graph, just the triples with ill-typed literals <Dominik_T> +1 for SHOULD Souri: the earlier the problems can be pointed out the better <ktk> Strawpoll: "Implementations MUST accept ill-typed literals" gets changed to "Implementations SHOULD accept ill-typed literals" <Dominik_T> +1 <gtw> +1 <ktk> +1 <pfps> +1 <AndyS> +1 <Souri> +1 <gkellogg> +1 <AZ> -0.3141592 <enrico> 0 <TallTed> -0.5 <james> 0 <niklasl> +0.5 (I might prefer some "SHOULD by default, MUST if asked to accept"...) TallTed: if we make this change, we have to be really clear how errors are dealt with AndyS: I don't think we should go into how errors and warnings are handled <TallTed> An ill typed literal is not a syntax error. <TallTed> An ill typed literal conforms to syntax. <Souri> +1 to AndyS AndyS: there's an historical example (??) where specs mentioned what to do with errors and it took a large space, and was eventually dropped niklasl: I had experienced cases of systems that reject things that I would have like be accepted because things evolved ktk: there could be a note that explain what pitfalls etc occur and how to deal with them |
A message, https://lists.w3.org/Archives/Public/public-rdf-star-wg/2024Nov/0008.html, was sent to the WG mailing list with a proposal to resolve this issue. The contents of the message are: PROPOSAL: Change the requirements for handling ill-typed literals so that The relevant wording is in RDF 1.2 Concepts and Syntax: If the literal's datatype IRI is in the set of recognized datatype IRIs, let d A possible change if the proposal is accepted is: ... |
The note in the possible wording above should be expanded to read: NOTE: Implementations MUST NOT exclude triples that do not contain literals with datatypes that are in their recognized datatypes from the RDF graphs they produce. |
This seems to be contradictory. Is "accept" meaning "can parse and continue" here? and then "produce ..." needs changing. Or is the MUST supposed to be SHOULD? "Implementations SHOULD accept ill-typed literals" |
@afs I don't think so. Implementations MUST accept ill-typed literals, in that they MUST NOT halt when they encounter an ill-typed literal, but they MAY decide to not include triples with ill-typed literals in the RDF graphs they produce. |
I agree with @afs that this can seem contradictory. I read the "produce RDF graphs for inputs" in
as requiring a conforming application to be able to return the graph with all its ill-typed literals intact, or otherwise it can't claim to be returning the original graph. Any other graph it produces, e.g. one in which ill-typed literals are dropped, can not claim to be the original graph. I read the second sentence
as honoring the fact that implementations are free to change graphs according to their needs. That is evident, but then the proposal adds the requirement that implementations SHOULD warn users if they drop ill-formed literals, and that seems sensible. If my interpretation is correct I support the design of the proposal, but maybe the wording could be made clearer? |
i suggest that the proposed change not be made for at least these reasons:
|
I don't see much loss in having literals like "a"^^xsd:int not being preserved. |
And |
It's good that you don't have such data to deal with. That's no guarantee that such data will never be encountered. What if the (apparently What if the apparent ill-typing is something like (I think) I'm fine with an implementation declaring that such data will be handled in that way, especially if some kind of alert is raised when it happens. I don't think I'm OK with our spec dictating any of the above handlings. Significantly, this would break from RDF 1.1 and 1.0, and I daresay, some datasets and stores would become non-compliant. I think there is little difference between rejecting literal data that doesn't match my internal definition of its declared data type, and rejecting literal data that has a declared data type that I don't recognize. I believe both should be accepted and stored. Errors may arise when some comparison function is applied to the literal based on its declared type and which fails because the literal does not actually fit that declared type. That is OK! This is the point at which the user may decide to change the type of that literal, or change the literal to suit the type, or some combination of the two. |
Section in current version of RDF 1.2 Concepts and Abstract Syntax: https://www.w3.org/TR/rdf12-concepts/#section-Graph-Literal It says:
The proposal is to include all possible variations being considered as comment here, so we can conduct a straw poll on them. |
Minimal change: Change to "Implementations SHOULD accept ..." where SHOULD is defined by RFC 2119:
This is then not a purely optional feature - that would use the word "MAY". |
I think that if we change this MUST to SHOULD, we should at least add a requirement that implementations MUST document their handling, probably to include some alert about dropping/ignoring, of such literals during LOAD, etc. |
This was discussed during the #rdf-star meeting on 05 December 2024. View the transcriptDrop the requirement to support ill-typed literals with recognized datatype IRIs 2ora: from outside the group ktk: last time (3 weeks ago), .... now we have a proposal. <ktk> w3c/rdf-concepts#60 (comment) <gb> Issue 60 Drop the requirement to support ill-typed literals with recognized datatype IRIs (by wouterbeek) [needs discussion] [spec:enhancement] james: as a matter of process, why is this being worked on? pchampin: W3C hat off -- while there was no errata, it seems a low hanging fruit and some existing implementation implement "should" - codify existing practice. <pchampin> I still need to write down an answer, but I have some data about that james: I asked for a catalog of implementations pchampin: I have data james: I am concerned because of interoperability ora: table until next time (US "table" meaning) gkellogg: it relates to "supported datatypes" and hence for the instance and interoperability is not related to "supported datatypes" tallted: not an erratum until previous work reviewed pchampin: not all system do the same james: could interpret it as an errata in the other direction <pchampin> +1 to provide a way for implementation to be explicit about what they are doing with ill-formed literals ora: suggest we see what data pchampin has collected <Zakim> TallTed, you wanted to ask about application-specific datatypes, which might be subtypes (or supertypes) of xsd TallTed: at the moment, anyone can define a datatype <niklasl> https://www.w3.org/TR/rdf12-concepts/#dfn-recognized-datatype-iri AndyS: the text refers to "recognized datatypes" that a system must declare. <ora> STRAWPOLL: table ( <gb> Issue 60 Drop the requirement to support ill-typed literals with recognized datatype IRIs (by wouterbeek) [needs discussion] [spec:enhancement] <ora> STRAWPOLL: table ( <gtw> +1 <ora> +1 <gkellogg> +1 <tl> +1 <pchampin> +1 <ktk> +1 <Souri> +1 <james> +1 <niklasl> +1 <TallTed> +1 <AndyS> +1 <doerthe> +1 <william_vw> +1 <AZ> +1 <eBremer> +1 <Tpt> +1 <olaf> +1 |
As a quick way of testing how different implementations handle ill-typed literals (for recognized datatypes), I tested the following SPARQL query on different SPARQL endpoint / libraries: SELECT ("a"^^<http://www.w3.org/2001/XMLSchema#integer> as ?x) {} The following implementation return
The following implementations raise an error
I'm not able to test on other implementations at this moment, but if others can test on Amazon Neptune, Oxigraph, others... I'm interested. edited thanks to @lisp for providing more results |
Useful background.
The only spec constraint is not redefining Unless a system advertises (in its documentation?) the set of recognized datatypes, we have to presume there aren't any. A web search didn't turn up any hits for recognized datatypes. Any "MUST" or "SHOULD" is on the elements of the set |
Since you included version info for (almost) all the other engines, please note that the Virtuoso instance currently hosting DBpedia is version |
Dydra $ curl https://dydra.com/nlv01111/gsp/sparql \
-H "Accept: application/sparql-results+json" \
-H "Content-Type: application/sparql-query" \
--data-binary 'SELECT ("a"^^<http://www.w3.org/2001/XMLSchema#integer> as ?x) {}' \
-is
HTTP/1.1 200 OK
Server: nginx/1.23.3
Date: Sat, 07 Dec 2024 12:55:57 GMT
Content-Type: application/sparql-results+json; charset=UTF-8
Transfer-Encoding: chunked
Connection: keep-alive
Last-Modified: Tue, 27 Dec 2022 13:50:14 GMT
Etag: 6BAC9724-9A0B-5941-8ADA-3B4C44295565
Vary: Accept, Accept-Datetime, Accept-Encoding, Origin, Revision
Accept-Ranges: none
Cache-Control: private
Request-Id: 99E41F7C-B49A-11EF-BF49-F02F7494A8ED
Strict-Transport-Security: max-age=31536000
Access-Control-Max-Age: 86400
Access-Control-Allow-Credentials: true
Access-Control-Allow-Methods: *
Access-Control-Allow-Origin: *
Access-Control-Allow-Headers: Accept, Accept-Asynchronous, Accept-Datetime, Asynchronous-Content-Type, Asynchronous-Location, Asynchronous-Method, Authorization, Content-Disposition, Content-Encoding, Content-Type, Etag, Graph, Introspection-Content-Type, Link, Location, Revision, X-Requested-With
Access-Control-Expose-Headers: *
{ "head": { "vars": [ "x" ] },
"results": {
"bindings": [
{ "x": {"type":"literal", "datatype":"http://www.w3.org/2001/XMLSchema#integer", "value":"a"} } ] } } |
Neptune (1.2.1.0) $ curl "https://${NEPTUNE_HOST}:8182/sparql" \
--aws-sigv4 "aws:amz:eu-west-1:neptune-db" \
--user "${AWS_ACCESS_KEY_ID}:${AWS_SECRET_ACCESS_KEY}" \
-H "x-amz-security-token: ${AWS_SESSION_TOKEN}" \
-H "Accept: application/sparql-results+json" \
-H "Content-Type: application/sparql-query" \
--data-binary 'SELECT ("a"^^<http://www.w3.org/2001/XMLSchema#integer> as ?x) {}' \
-is
HTTP/1.1 200 OK
transfer-encoding: chunked
content-type: application/sparql-results+json; charset=UTF-8
{
"head" : {
"vars" : [ "x" ]
},
"results" : {
"bindings" : [ {
"x" : {
"datatype" : "http://www.w3.org/2001/XMLSchema#integer",
"type" : "literal",
"value" : "a"
}
} ]
}
} |
Blazegraph returns an error: https://w.wiki/CM8o Oxigraph returns |
More background:
|
An alternative approach is to put the "recognized dataypes" concept into RDF Semantics.
Idea: define "RDF Processor" as a system that implements RDF Semantics (and not simple semantics). We could have, informally, RDFS processor, D-entailment processor, simple processor etc. This would go in RDF Semantics where it is needed. Remove the "recognized datatype" content from RDF Concepts. SPARQL is not affected nor is any SPARQL implementation's handling of datatypes. |
Why do we need to continue this discussion?
|
Recognized datatype comes from RDF MT (Semantics) datatype maps. RDF New , section "4. Semantics"
|
I don't think the SPARQL test alone is accurate. Stardog seems to accept it in SPARQL, but by default it will refuse it on loading, as "strict parsing" is true by default. See docs here https://docs.stardog.com/operating-stardog/database-administration/database-configuration#strictparsing |
Other tests would be: PREFIX xsd: <http://www.w3.org/2001/XMLSchema#>
SELECT ("500"^^xsd:byte as ?x) {}
To see if there is a numeric value: PREFIX xsd: <http://www.w3.org/2001/XMLSchema#>
SELECT ("500"^^xsd:byte - 400 as ?x2) {} |
This was discussed during the #rdf-star meeting on 19 December 2024. View the transcriptDrop the requirement to support ill-typed literals with recognized datatype IRIs 3ktk: what is the conclusion on this one? AndyS: I dug around in the history a bit. Found that rdf 1.1 mentions this. gkellogg: we're not modifying 1.1 semantics. AndyS: we're keeping it in 1.2 semantics, but not having it in 1.2 concepts. <Zakim> pfps, you wanted to ask whether we are talking about required or recognized data types pfps: I'm confused. Are we talking about recognized or required datatypes. AndyS: phrase is "required" pfps: items have "recognized". AndyS: should have said "recognized". semantics does put requirements on the "recognized" set. pfps: not sure why that is relevant. AndyS: I am suggesting taking it out of concepts. Seems to have no purpose except to support extensions. pfps: why is that relevant to discussion of "recognized" datatypes? pchampin: the normative statement we are discussing is from rdf concepts. ora2: the only required datatypes are string and something else...? AndyS: in rdf semantics, string and langString. I presume we would add langString as well. gkellogg: as well as rdf:HTML AndyS: and a few others. those were never required. james: I'm trying to understand the consequence of removing the requirement for recognized datatypes. AndyS: no, because in RDF concepts, we only have datatype terms. we don't have any meaning to the datatypes. gkellogg: I think it comes down to XML scheme built-in datatypes are recommended but not required in concepts. AndyS: xsd datatypes are really there for different purpose. pchampin: looking at rdf 1.1 semantics, I seem to remember there was something that gave some implementation leeway when the graph is unsatisfiable. <niklasl> "If the literal is ill-typed then the L2V(I(aaa)) mapping has no value, and so the literal cannot denote anything. In this case, any triple containing the literal must be false. Thus, any triple, and hence any graph, containing an ill-typed literal will be D-unsatisfiable, i.e. false in every D-interpretation." from <niklasl> https://www.w3.org/TR/rdf12-semantics/#D_interpretations pchampin: I think that makes sense. Somehow permited by rdf semantics at this point, so why should it be forbidden by concepts. pfps: in rdf 1.1 semantics, there was a change. pchampin: maybe this statement in concepts in related to that in 1.0. ora2: this makes me think since this is hard for us, we should somehow spell out exactly what it is that we are hoping/expecting implementations to do. AndyS: my hope is it makes no difference. ora2: how are implementations supposed to declare this? AndyS: we don't have mechanisms to do that. ora2: only remotely remeniscent mechanism is SPARQL implementations disclosing entailment regimes. AndyS: most in SPARQL happens because you perform arithmatic on datatypes. rules from F&O applies even when you're in simple entailment. ora2: let's for a moment ignore that we don't have such a mechanism. pretend there is one. we should have a clear recipe for implementations as to how they should behave. AndyS: at the moment, says "MUST accept ill-defined literals". <AZ> Re. what pchampin said: In RDF 1.1 Semantics, Section 7.2 says "RDF processors MAY treat an unsatisfiable graph as signaling an error condition, but this is not required." <pchampin> https://www.w3.org/TR/rdf11-semantics/#D_entailment pchampin: I found what I had remembered. section 7.2. ora2: that and the fact that ill-formed literals lead to unsatisfiable graph. pchampin: yes. since rdf 1.1. james: MAY do that if they're doing D-entailment is what you read. pchampin: only makes sense with D-entailment. james: not required anywhere that they do D-entailment. pchampin: I agree. The phrasing refers to "recognized types", which I think is only defined with respect to D-entailment. ora2: we would do a service to the community if we spelled this out for people. AndyS: if we remove it, we're doing what everyone has been doing up to now. ora2: not only spell it out but define anomalous behavior. AndyS: nowhere in the specs touches on this as a round-trip requirement. james: the simplest simplistic formulation might be if you are doing D-entailment, you MUST reject. If not, you MUST accept. AZ: I think what RDF concepts and semantics say is not necessarily inconsistent. <pchampin> james, I'm not sure I like "improving expectations" if we envision that tools won't meet these expectations AZ: example of ontologies in OWL can be inconsistent. You can have tools that open inconsistent ontologies like Protoge. <pfps> I agree with Antoine, actually the reasoning might not even signal an "error" but just complain about inconsistency and proceed. AZ: parser should handle any literal. but if you want to do some processes later on the data you parsed, maybe can raise an error. but that's at a different level. ora2: so a layered approach. parsing can turn into a graph, but can't do whatever reasoning you want. different things. <pchampin> +1 AZ, the problem of this MUST is that it is not granular enough ora2: still, I think we need to be clear on this and spell it out. AndyS: don't we have the structure for saying that? Isn't that simple entailment? ora2: if you don't do D-entailment, isn't that what we conluded a moment ago? AndyS: everything must be build on simple entailment. that defines truth. pchampin: I really like AZ's suggestion. niklasl: I agree with that from user perspective. <james> pchampin: clarity as to which component supports what would reduce ambiguity. niklasl: if I had system with entailment built in. If I know that my implementation has D-entailment. <pchampin> and that's where "SHOULD support" would leave it to implementers discretion niklasl: more problem having it keep around boolean values with invalid values. not the contract I expect it to fill. ora2: I will try to take a stab at writing this out in an understandable way. AndyS: related thing is are we going to take it out of concepts? leave it just in semantics. ora2: I don't know how to answer that. Think we need a paragraph or two somewhere that spells it out. <TallTed> I think moving it to semantics makes most sense <TallTed> yes, spelled out better would also be good. :-) I'll try to help with text once it hits a PR. <niklasl> +1 |
RDF-Semantics does not reference RDF-Concepts for recognized datatypes. |
Observation
RDF 1.1 requires that implementations support ill-typed literals, including ill-typed literals with recognized datatype IRIs.
Ill-typed literals with recognized datatype IRIs do not have any known use cases. They are semantically inconsistent, do not denote anything, have no value, and any triple that contains them is false in every interpretation.
Notice that there is nothing wrong with requiring implementations to support ill-typed literals with unrecognized datatype IRIs. For example, it is good that RDF implementations are required to support literals like [1] that have a datatype IRI that is not broadly recognized.
However, it is unclear why implementations are allowed to support, let alone are required to support, ill-typed literals with recognized datatype IRIs.
Example
Suppose a triple store recognizes the RDF datatype IRIs + the XSD datatype IRIs + the GeoSPARQL datatype IRIs. Such a triple store can upon data ingest immediately detect that [2] and [3] are ill-typed literals with recognized datatype IRI.
The RDF 1.1 standard forbids triple stores to throw an error upon encountering data that contains [2] or [3], even though this may be the preferred data quality approach for many users.
Suggestion
In RDF 1.2, let's weaken the RDF 1.1 phrase "Implementations MUST accept ill-typed literals" to:
Implementations MUST support the RDF datatype IRIs, and MAY support any other datatype IRIs that they believe important enough for their users. The notion "recognized datatype IRI" is used as defined in RDF 1.1 Semantics.
Ramifications
The proposed change makes it possible for RDF 1.2 data to be accepted in one implementation, but not in another implementation. For example, it is possible to upload data that contains literals [2] and [3] into an implementation that does not recognize the
xsd:boolean
datatype IRI. But it is not possible to upload the same data into an implementation that does recognize thexsd:boolean
datatype IRI.This differentiation is a good thing, because it allows stricter implementations to be created, rather than requiring all implementations to support the exact same ill-typed nonsense data.
Notice that RDF 1.1 Semantics already allows implementations to differ from one another in their support for more/fewer recognized datatype IRIs. Implementations that differ in their recognized datatype IRIs already differ in their behavior in RDF 1.1.
The text was updated successfully, but these errors were encountered: