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

The result of property-accessing expression is incorrect #5226

Closed
czpmango opened this issue Jan 9, 2023 · 5 comments · Fixed by #5230
Closed

The result of property-accessing expression is incorrect #5226

czpmango opened this issue Jan 9, 2023 · 5 comments · Fixed by #5230
Assignees
Labels
affects/none PR/issue: this bug affects none version. process/done Process of bug severity/none Severity of bug type/bug Type: something is unexpected
Milestone

Comments

@czpmango
Copy link
Contributor

czpmango commented Jan 9, 2023

Describe the bug (required)
As title.

Your Environments (required)
nebula v3.3.0

How To Reproduce(required)

(root@nebula) [nba]> match (v:player)-[e]->(n) with {key:v} as m return m.key.player.age limit 3
+------------------+
| m.key.player.age |
+------------------+
| __NULL__         |
| __NULL__         |
| __NULL__         |
+------------------+
Got 3 rows (time spent 17007/17372 us)

Expected behavior
Get the correct property.

@czpmango czpmango added the type/bug Type: something is unexpected label Jan 9, 2023
@czpmango czpmango self-assigned this Jan 9, 2023
@github-actions github-actions bot added affects/none PR/issue: this bug affects none version. severity/none Severity of bug labels Jan 9, 2023
@czpmango
Copy link
Contributor Author

czpmango commented Jan 9, 2023

#5054

@czpmango
Copy link
Contributor Author

czpmango commented Jan 9, 2023

We should consider another way to fix this issue #5051

@xtcyclist xtcyclist added this to the v3.5.0 milestone Jan 10, 2023
@xtcyclist
Copy link
Contributor

xtcyclist commented Jan 10, 2023

The unwind problem as reported in #5051 remain fixed by rewriting the expression. Other issues (e.g., ListComprehension) fixed by #5054 are going to be reopened by #5229.

@xtcyclist
Copy link
Contributor

xtcyclist commented Jan 10, 2023

      MATCH p = (n:player{name:"LeBron James"})-[:like]->(m)
      RETURN nodes(p)[0].player.age AS age1,
             nodes(p)[1].player.age AS age2,
             reduce(x = 10, n IN nodes(p) | n.player.age - x) AS x

While fixing this issue, please take a note that queries like the above need to be ok to run.

And need to reopen Attribute1.feature.

@czpmango
Copy link
Contributor Author

MATCH p = (n:player{name:"LeBron James"})-[:like]->(m)
RETURN nodes(p)[0].player.age AS age1,
nodes(p)[1].player.age AS age2,
reduce(x = 10, n IN nodes(p) | n.player.age - x) AS x

Ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects/none PR/issue: this bug affects none version. process/done Process of bug severity/none Severity of bug type/bug Type: something is unexpected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants