GraphScope icon indicating copy to clipboard operation
GraphScope copied to clipboard

[BUG] Dynamic params are not passed to physical layer

Open zhanglei1949 opened this issue 1 year ago • 2 comments

MATCH (p: PERSON{id: $personId}) -[k:KNOWS*1..4]-(f: PERSON {firstName: $firstName})
WHERE p <> f
with f AS f, length(k) as len
with f AS f, min(len) as distance
ORDER  BY distance ASC, f.lastName ASC, f.id ASC
LIMIT 20

For example, When using GraphPlanner to generate the physical plan for this query, only one dynamic param presents. 1 firstName, the $personid is missing.

zhanglei1949 avatar Mar 27 '24 05:03 zhanglei1949

To reproduce, use the following config

graph.planner.opt: RBO
physical.opt.config: ffi
graph.planner.rules: FilterIntoJoinRule, FilterMatchRule, NotMatchToAntiJoinRule

zhanglei1949 avatar Mar 27 '24 06:03 zhanglei1949

/cc @BingqingLyu, this issus/pr has had no activity for for a long time, could you folks help to review the status ? To suppress further notifications,

  • for issues,
    • if it is waiting for further response from the reporter/author, please help to add the label requires-further-info,
    • if you have already started working on it, please add the label work-in-progress to the issue,
    • if this issue requires further designing discussion and not in current plan, or won't be fixed, please add the label requires-further-discussion or wontfix to the issue,
  • for pull requests,
    • if you are still working on it and it is not ready for reviewing, please convert this pull request as draft PR,
    • if you have decided to hold this development on, please add the requires-further-discussion label to the pull request. Thanks!

github-actions[bot] avatar Apr 12 '24 00:04 github-actions[bot]