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

Unable to query based on the list of vi'd's returned by the split function #4417

Closed
shanlai opened this issue Jul 14, 2022 · 0 comments · Fixed by #4517
Closed

Unable to query based on the list of vi'd's returned by the split function #4417

shanlai opened this issue Jul 14, 2022 · 0 comments · Fixed by #4517
Assignees
Labels
severity/minor Severity of bug type/bug Type: something is unexpected
Milestone

Comments

@shanlai
Copy link

shanlai commented Jul 14, 2022

Describe the bug (required)
As title

Your Environments (required)
image

How To Reproduce(required)

  1. execute nGQL
MATCH (v:player)--(v2) WHERE id(v2) IN ["player101","player102"] and v2.player.age>34  RETURN v2;
RETURN split("player101,player102", ",");
MATCH (v:player)--(v2) WHERE id(v2) IN split("player101,player102", ",") and v2.player.age>34  RETURN v2;

image

  1. Execution plan
    nGQL:MATCH (v:player)--(v2) WHERE id(v2) IN ["player101","player102"] and v2.player.age>34 RETURN v2;
<style> </style>
id name dependencies profiling data operator info
13 Project 10   outputVar: {  "colNames": [    "v2"  ],  "type": "DATASET",  "name": "__Project_8"}inputVar: __Filter_10columns: [  "$-.v2"]
10 Filter 5   outputVar: {  "colNames": [    "v2",    "__VAR_0",    "v"  ],  "type": "DATASET",  "name": "__Filter_10"}inputVar: __AppendVertices_5condition: ((id($-.v2) IN ["player101","player102"]) AND (-.v2.player.age>34))isStable: false
5 AppendVertices 4   outputVar: {  "colNames": [    "v2",    "__VAR_0",    "v"  ],  "type": "DATASET",  "name": "__AppendVertices_5"}inputVar: __Traverse_4space: 3dedup: truelimit: -1filter: orderBy: []src: none_direct_dst($-.__VAR_0)props: [  {    "props": [      "_tag"    ],    "tagId": 4  }]exprs: vertex_filter: player._tag IS NOT EMPTYif_track_previous_path: true
4 Traverse 2   outputVar: {  "colNames": [    "v2",    "__VAR_0"  ],  "type": "DATASET",  "name": "__Traverse_4"}inputVar: __Dedup_2space: 3dedup: truelimit: -1filter: orderBy: []src: $-._videdgeTypes: []edgeDirection: BOTHvertexProps: [  {    "props": [      "name",      "age",      "_tag"    ],    "tagId": 4  },  {    "props": [      "name",      "_tag"    ],    "tagId": 5  }]edgeProps: [  {    "props": [      "_dst",      "_rank",      "_type",      "_src"    ],    "type": "-7"  },  {    "props": [      "_dst",      "_rank",      "_type",      "_src"    ],    "type": "7"  },  {    "props": [      "_dst",      "_rank",      "_type",      "_src"    ],    "type": "-6"  },  {    "props": [      "_dst",      "_rank",      "_type",      "_src"    ],    "type": "6"  }]statProps: exprs: random: falsesteps: vertex filter: edge filter: if_track_previous_path: false
2 Dedup 1   outputVar: {  "colNames": [    "_vid"  ],  "type": "DATASET",  "name": "__Dedup_2"}inputVar: __VAR_1
1 PassThrough 3   outputVar: {  "colNames": [    "_vid"  ],  "type": "DATASET",  "name": "__VAR_1"}inputVar:
3 Start     outputVar: {  "colNames": [],  "type": "DATASET",  "name": "__Start_3"}

nGQL:MATCH (v:player)--(v2) WHERE id(v2) IN ["player101","player102"] and v2.player.age>34 RETURN v2;

<style> </style>
id name dependencies profiling data operator info
14 Project 10   outputVar: {  "colNames": [    "v2"  ],  "type": "DATASET",  "name": "__Project_8"}inputVar: __Filter_10columns: [  "$-.v2"]
10 Filter 5   outputVar: {  "colNames": [    "v",    "__VAR_0",    "v2"  ],  "type": "DATASET",  "name": "__Filter_10"}inputVar: __AppendVertices_5condition: ((id($-.v2) IN ["player101","player102"]) AND (-.v2.player.age>34))isStable: false
5 AppendVertices 4   outputVar: {  "colNames": [    "v",    "__VAR_0",    "v2"  ],  "type": "DATASET",  "name": "__AppendVertices_5"}inputVar: __Traverse_4space: 3dedup: truelimit: -1filter: orderBy: []src: none_direct_dst($-.__VAR_0)props: [  {    "props": [      "name",      "age",      "_tag"    ],    "tagId": 4  },  {    "props": [      "name",      "_tag"    ],    "tagId": 5  }]exprs: vertex_filter: if_track_previous_path: true
4 Traverse 12   outputVar: {  "colNames": [    "v",    "__VAR_0"  ],  "type": "DATASET",  "name": "__Traverse_4"}inputVar: __Filter_2space: 3dedup: truelimit: -1filter: orderBy: []src: $-._videdgeTypes: []edgeDirection: BOTHvertexProps: [  {    "props": [      "_tag"    ],    "tagId": 4  }]edgeProps: [  {    "props": [      "_dst",      "_rank",      "_type",      "_src"    ],    "type": "-7"  },  {    "props": [      "_dst",      "_rank",      "_type",      "_src"    ],    "type": "7"  },  {    "props": [      "_dst",      "_rank",      "_type",      "_src"    ],    "type": "-6"  },  {    "props": [      "_dst",      "_rank",      "_type",      "_src"    ],    "type": "6"  }]statProps: exprs: random: falsesteps: vertex filter: player._tag IS NOT EMPTYedge filter: if_track_previous_path: false
12 ScanVertices 3   outputVar: {  "colNames": [    "_vid",    "player._tag"  ],  "type": "DATASET",  "name": "__Filter_2"}inputVar: space: 3dedup: falselimit: -1filter: player._tag IS NOT EMPTYorderBy: []props: [  {    "props": [      "_tag"    ],    "tagId": 4  }]exprs:
3 Start     outputVar: {  "colNames": [],  "type": "DATASET",  "name": "__Start_3"}

Expected behavior

Additional context

@shanlai shanlai added the type/bug Type: something is unexpected label Jul 14, 2022
@Sophie-Xie Sophie-Xie added this to the v3.3.0 milestone Jul 14, 2022
Sophie-Xie pushed a commit that referenced this issue Aug 15, 2022
* Add case for bugfix

* Add explain
@jinyingsunny jinyingsunny added the severity/minor Severity of bug label Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity/minor Severity of bug type/bug Type: something is unexpected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants