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

Disable setting the IRI of predicates in Relationship tables of RDFGraphs or make it behave correctly #2773

Closed
semihsalihoglu-uw opened this issue Feb 2, 2024 · 2 comments
Assignees
Labels
bug Something isn't working high-priority rdf Issues related to RDF support

Comments

@semihsalihoglu-uw
Copy link
Contributor

I tried the following query in the UniKG example in the documentation:

MATCH (r:UniKG_r {iri:"http://kuzu.io/rdf-ex#Waterloo"})-[p:UniKG_lt]->(o) 
WHERE p.iri =  "http://kuzu.io/rdf-ex#population" 
SET p.iri = "http://kuzu.io/rdf-ex#population2" 
RETURN r.iri, p.iri, o.iri

This query is trying to set the IRI of the resource node. I got the following output:

--------------------------------------------------
| r.iri                          | p.iri | o.val |
--------------------------------------------------
| http://kuzu.io/rdf-ex#Waterloo |       | 60000 |
--------------------------------------------------

Which already is wrong. But it also did not have any effect, i.e., the IRI stayed the same:

MATCH (r:UniKG_r {iri:"http://kuzu.io/rdf-ex#Waterloo"})-[p:UniKG_lt]->(o) 
WHERE p.iri =  "http://kuzu.io/rdf-ex#population" 
RETURN r.iri, p.iri, o.val;
-----------------------------------------------------------------------------
| r.iri                          | p.iri                            | o.val |
-----------------------------------------------------------------------------
| http://kuzu.io/rdf-ex#Waterloo | http://kuzu.io/rdf-ex#population | 60000 |
-----------------------------------------------------------------------------

The previous IRI is still there. And when I query for all triples, I don't see any triple of Waterloo with predicate http://kuzu.io/rdf-ex#population2.

We should either disable setting iris of predicates or support this so that we implement the right behavior, which is:

  1. add the predicate if needed to Resources table
  2. set the IRI of the predicate relationship.
@semihsalihoglu-uw semihsalihoglu-uw added bug Something isn't working high-priority rdf Issues related to RDF support labels Feb 2, 2024
@semihsalihoglu-uw semihsalihoglu-uw mentioned this issue Feb 2, 2024
24 tasks
@semihsalihoglu-uw
Copy link
Contributor Author

As per our discussion, let's disable this. I'll document it accordingly.

@andyfengHKU
Copy link
Contributor

Fixed in PR #2786

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high-priority rdf Issues related to RDF support
Projects
None yet
Development

No branches or pull requests

2 participants