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

startVid info lost in certain multiple match/WITH case #4806

Closed
wey-gu opened this issue Oct 28, 2022 · 2 comments
Closed

startVid info lost in certain multiple match/WITH case #4806

wey-gu opened this issue Oct 28, 2022 · 2 comments
Assignees
Labels
affects/none PR/issue: this bug affects none version. need to discuss Solution: issue or PR without a clear conclusion on whether to handle it process/done Process of bug severity/minor Severity of bug type/bug/compatibility Bugs related to compatibility with other languages like openCypher. type/bug Type: something is unexpected wontfix Solution: this will not be worked on recently
Milestone

Comments

@wey-gu
Copy link
Contributor

wey-gu commented Oct 28, 2022

Describe the bug (required)

# OK, passthrough
match (m:`player`) where id(m) in ["player100", "player101", "player102"]
with m
MATCH (x)-[r1:follow]-(n:`player`)
WHERE id(x) == "player100" AND r1.degree>10 AND NOT (m==n)
RETURN n


# NOK, but still goes to indexscan

match (m:`player`) where id(m) in ["player100", "player101", "player102"]
WITH collect(distinct m)[0..10] as col_appl_1st_v4,m
UNWIND col_appl_1st_v4 as x
OPTIONAL MATCH (x)-[r1:follow]-(n:`player`)
RETURN n

# same as above
match (m:`player`) where id(m) in ["player100", "player101", "player102"]
WITH collect(distinct m)[0..10] as col_appl_1st_v4,m
UNWIND col_appl_1st_v4 as x_node
OPTIONAL MATCH (x)-[r1:follow]-(n:`player`)
WHERE id(x) == id(x_node)
RETURN n

# NOK, ask for indexscan

match (m:`player`) where id(m) in ["player100", "player101", "player102"]
WITH collect(distinct m)[0..10] as col_appl_1st_v4,m
UNWIND col_appl_1st_v4 as x
OPTIONAL MATCH (x)-[r1:follow]-(n:`player`)
WHERE r1.degree>10
RETURN n

Your Environments (required)

  • OS: uname -a
  • Compiler: g++ --version or clang++ --version
  • CPU: lscpu
  • Commit id (e.g. a3ffc7d8)

How To Reproduce(required)

Steps to reproduce the behavior:

as above

Expected behavior

This should be passthrough:

OPTIONAL MATCH (x)-[r1:follow]-(n:`player`)

Additional context

@wey-gu wey-gu added the type/bug Type: something is unexpected label Oct 28, 2022
@wey-gu wey-gu changed the title startVid info lost in certain multiple match case startVid info lost in certain multiple match/WITH case Oct 28, 2022
@Sophie-Xie Sophie-Xie modified the milestones: v3.3.0, v3.4.0 Oct 31, 2022
@xtcyclist
Copy link
Contributor

There are multiple issues related to optional match. Maybe we shall try considering them altogether. For example, #4745.

@xtcyclist xtcyclist added the type/bug/compatibility Bugs related to compatibility with other languages like openCypher. label Nov 9, 2022
@jinyingsunny jinyingsunny added the severity/minor Severity of bug label Nov 10, 2022
@caton-hpg caton-hpg added the wontfix Solution: this will not be worked on recently label Nov 18, 2022
@Sophie-Xie Sophie-Xie assigned MuYiYong and unassigned caton-hpg Dec 1, 2022
@HarrisChu HarrisChu added the affects/none PR/issue: this bug affects none version. label Dec 1, 2022
@Sophie-Xie Sophie-Xie added the need to discuss Solution: issue or PR without a clear conclusion on whether to handle it label Dec 5, 2022
@xtcyclist
Copy link
Contributor

Close optional match realted issues.

@github-actions github-actions bot added the process/fixed Process of bug label Dec 21, 2022
@Hester-Gu Hester-Gu added the process/done Process of bug label Jan 13, 2023
@github-actions github-actions bot removed the process/fixed Process of bug label Jan 13, 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. need to discuss Solution: issue or PR without a clear conclusion on whether to handle it process/done Process of bug severity/minor Severity of bug type/bug/compatibility Bugs related to compatibility with other languages like openCypher. type/bug Type: something is unexpected wontfix Solution: this will not be worked on recently
Projects
None yet
Development

No branches or pull requests

8 participants