fix match multiple id error #3899
Conversation
nevermore3
added
cherry-pick-v3.0
| When executing query: | ||
| """ | ||
| MATCH (a)-[e:like]->(b) | ||
| WHERE id(a) == 'Tim Duncan' OR id(b) == 'Tony Parker' |
There was a problem hiding this comment.
This should report error
There was a problem hiding this comment.
use like index not vertexIDseek
There was a problem hiding this comment.
Emm. It's not related to this VIdSeek, better not put in this file.
There was a problem hiding this comment.
And you should add fail case of or expression.
There was a problem hiding this comment.
Emm. It's not related to this VIdSeek, better not put in this file.
it's ok to put this case here
| When executing query: | ||
| """ | ||
| MATCH (a)-[e:like]->(b) | ||
| WHERE id(a) == 'Tim Duncan' OR id(b) == 'Tony Parker' |
There was a problem hiding this comment.
Should add integer vid test case too.
There was a problem hiding this comment.
ok， add case in next pr
There was a problem hiding this comment.
Why don't add in this PR directly?
There was a problem hiding this comment.
@Sophie-Xie waiting for this PR to publish test version
Co-authored-by: jimingquan <[email protected]>
What type of PR is this?
What problem(s) does this PR solve?
Issue(s) number:
Description:
How do you solve it?
Special notes for your reviewer, ex. impact of this fix, design document, etc:
Checklist:
Tests:
Affects:
Release notes:
Please confirm whether to be reflected in release notes and how to describe: