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

Bug: MATCH path with recursive relation returns nodes/rels in wrong order #3785

Closed
Tracked by #3666
sapalli2989 opened this issue Jul 9, 2024 · 3 comments · Fixed by #3799
Closed
Tracked by #3666

Bug: MATCH path with recursive relation returns nodes/rels in wrong order #3785

sapalli2989 opened this issue Jul 9, 2024 · 3 comments · Fixed by #3799
Assignees
Labels
bug Something isn't working

Comments

@sapalli2989
Copy link
Contributor

Kùzu version

0.4.2

What operating system are you using?

Fedora 39

What happened?

MATCHing all paths from any user to :City {name: "Guelph"} via

MATCH p = (u:User)-[e*]->(c:City {name: "Guelph"}) 
RETURN properties(nodes(p),'name'), properties(rels(p),'_label');

returns

["Noura","Guelph"]	["LivesIn"]
["Zhang","Noura","Guelph"]	["LivesIn","Follows"]
["Adam","Noura","Zhang","Guelph"]	["LivesIn","Follows","Follows"]
["Karissa","Noura","Zhang","Guelph"]	["LivesIn","Follows","Follows"]
["Adam","Noura","Zhang","Karissa","Guelph"]	["LivesIn","Follows","Follows","Follows"]

, which doesn't reflect nodes and rel order in graph:
screenshot

(demo dataset)

Are there known steps to reproduce?

No response

@sapalli2989 sapalli2989 added the bug Something isn't working label Jul 9, 2024
@prrao87
Copy link
Member

prrao87 commented Jul 9, 2024

Hi @sapalli2989, I think this is related to #3751 and #3169. I believe the fix was issued on the latest dev build as per #3765. Could you try it there and let us know?

@sapalli2989
Copy link
Contributor Author

@prrao87 Just tried out with latest nightly cli from https://github.com/kuzudb/kuzu/actions/runs/9853279670 . Result is unchanged 🤔:

kuzu> MATCH p = (u:User)-[e*]->(c:City {name: "Guelph"})
..> RETURN properties(nodes(p),'name'), properties(rels(p),'_label');
-------------------------------------------------------------------------
| PROPERTIES(NODES(p),name)         | PROPERTIES(RELS(p),_label)        |
-------------------------------------------------------------------------
| [Noura,Guelph]                    | [LivesIn]                         |
-------------------------------------------------------------------------
| [Zhang,Noura,Guelph]              | [LivesIn,Follows]                 |
-------------------------------------------------------------------------
| [Adam,Noura,Zhang,Guelph]         | [LivesIn,Follows,Follows]         |
-------------------------------------------------------------------------
| [Karissa,Noura,Zhang,Guelph]      | [LivesIn,Follows,Follows]         |
-------------------------------------------------------------------------
| [Adam,Noura,Zhang,Karissa,Guelph] | [LivesIn,Follows,Follows,Follows] |
-------------------------------------------------------------------------

@prrao87
Copy link
Member

prrao87 commented Jul 10, 2024

Ok, this is something worth keeping open then.

@andyfengHKU andyfengHKU mentioned this issue Jul 10, 2024
81 tasks
@andyfengHKU andyfengHKU self-assigned this Jul 10, 2024
@andyfengHKU andyfengHKU mentioned this issue Jul 11, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants