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

Add more pattern expression test cases #5200

Open
czpmango opened this issue Jan 5, 2023 · 3 comments
Open

Add more pattern expression test cases #5200

czpmango opened this issue Jan 5, 2023 · 3 comments
Assignees
Labels
affects/none PR/issue: this bug affects none version. severity/none Severity of bug type/bug Type: something is unexpected

Comments

@czpmango
Copy link
Contributor

czpmango commented Jan 5, 2023

  
(root@nebula) [nba]> MATCH (v:player{name: 'Tim Duncan'})-[e:like*1..3]->(n), (t:team {name: "Spurs"}) WITH v, e, collect(distinct n) AS ns UNWIND [n in ns | ()-[e*2..4]->(n:player)] AS p RETURN p
+-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| p                                                                                                                                                                                                                                                                                       |
+-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| []                                                                                                                                                                                                                                                                                      |
| []                                                                                                                                                                                                                                                                                      |
| []                                                                                                                                                                                                                                                                                      |
| [<("Tony Parker" :player{age: 36, name: "Tony Parker"})<-[:like@0 {likeness: 95}]-("Tim Duncan" :player{age: 42, name: "Tim Duncan"} :bachelor{name: "Tim Duncan", speciality: "psychology"})<-[:like@0 {likeness: 95}]-("Tony Parker" :player{age: 36, name: "Tony Parker"})>]         |
| []                                                                                                                                                                                                                                                                                      |
| []                                                                                                                                                                                                                                                                                      |
| [<("Manu Ginobili" :player{age: 41, name: "Manu Ginobili"})<-[:like@0 {likeness: 95}]-("Tim Duncan" :player{age: 42, name: "Tim Duncan"} :bachelor{name: "Tim Duncan", speciality: "psychology"})<-[:like@0 {likeness: 90}]-("Manu Ginobili" :player{age: 41, name: "Manu Ginobili"})>] |
| []                                                                                                                                                                                                                                                                                      |
| []                                                                                                                                                                                                                                                                                      |
| []                                                                                                                                                                                                                                                                                      |
| []                                                                                                                                                                                                                                                                                      |
+-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+

Expected behivor:
Consistent with neo4j.

@czpmango czpmango added the type/bug Type: something is unexpected label Jan 5, 2023
@github-actions github-actions bot added affects/none PR/issue: this bug affects none version. severity/none Severity of bug labels Jan 5, 2023
@Sophie-Xie Sophie-Xie added this to the v3.4.0 milestone Jan 10, 2023
@Sophie-Xie Sophie-Xie removed this from the v3.4.0 milestone Jan 29, 2023
@czpmango
Copy link
Contributor Author

Repeating variable length edge variables will not be supported in 3.x versions, just as our offline discussion.
@nevermore3 cc
Disable this feature for now.

@czpmango
Copy link
Contributor Author

Also related to #5221.

@Sophie-Xie Sophie-Xie added this to the v3.5.0 milestone Feb 27, 2023
@Sophie-Xie Sophie-Xie added need to discuss Solution: issue or PR without a clear conclusion on whether to handle it and removed need to discuss Solution: issue or PR without a clear conclusion on whether to handle it labels Mar 6, 2023
@Sophie-Xie
Copy link
Contributor

Repeating variable length edge variables will not be supported in 3.x versions, just as our offline discussion. @nevermore3 cc Disable this feature for now.

#5268

@Sophie-Xie Sophie-Xie removed this from the v3.5.0 milestone Mar 10, 2023
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. severity/none Severity of bug type/bug Type: something is unexpected
Projects
None yet
Development

No branches or pull requests

2 participants