-
Notifications
You must be signed in to change notification settings - Fork 430
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
feat(interactive): Support []
Operator in Cypher Queries
#3168
Conversation
Variable List
in Cypher QueriesVariable_List
Operator in Cypher Queries
Variable_List
Operator in Cypher Queries[]
Operator in Cypher Queries
@Test | ||
public void with_11_test() { | ||
RelNode project = | ||
Utils.eval("Match (a:person)-[]-(b:person) Return [a.name, b.age, 1]").build(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how about returning an empty array like [ ]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how about returning an empty array like
[ ]
We have the same behavior as official Neo4j, both of which output an empty list.
@neo4j> Match (a) Return [];
+-----+
| $f0 |
+-----+
| [] |
| [] |
| [] |
| [] |
| [] |
| [] |
+-----+
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3168 +/- ##
===========================================
- Coverage 73.25% 42.29% -30.96%
===========================================
Files 102 101 -1
Lines 11006 10907 -99
===========================================
- Hits 8062 4613 -3449
- Misses 2944 6294 +3350 see 62 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
What do these changes do?
variable list
operator which is denoted by[a.name, a.age]
in cypher queriesGraphPath
to differentiate it from other ArrayTypes, as thelength()
function can only be applied toGraphPath
typesRelated issue number
Fixes