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

test_single_shortestpaths3 fails with not found results #4424

Closed
xtcyclist opened this issue Jul 14, 2022 · 3 comments
Closed

test_single_shortestpaths3 fails with not found results #4424

xtcyclist opened this issue Jul 14, 2022 · 3 comments
Labels
severity/minor Severity of bug type/bug Type: something is unexpected
Milestone

Comments

@xtcyclist
Copy link
Contributor

Please check the FAQ documentation before raising an issue

Describe the bug (required)

E NotFoundRow: 4: |<("Tim Duncan":bachelor{name:Tim Duncan,speciality:psychology}:player{age:42,name:Tim Duncan})-[:serve@0{end_year:2016,start_year:1997}]->("Spurs":team{name:Spurs})<-[:serve@0{end_year:2019,start_year:2017}]-("Rudy Gay":player{age:32,name:Rudy Gay})-[:serve@0{end_year:2017,start_year:2013}]->("Kings":team{name:Kings})>|

Your Environments (required)

  • OS: ubuntu 20.04
  • Compiler: clang-10
  • CPU: lscpu
  • Commit id: 7981108

How To Reproduce(required)

Steps to reproduce the behavior:

  1. Step 1
    make RM_DIR=false DEBUG=false J=6 tck
    or
    pytest -k 'test_single_shortestpaths3' .
  2. Step 2
  3. Step 3

Expected behavior

Additional context

Not sure whether it's really a bug, or it's just the test results are not deterministic.

@xtcyclist
Copy link
Contributor Author

xtcyclist commented Jul 15, 2022

this turns out to be an expected result. 'match shortestPath()' returns only 1 matching result out of many, the contents of which depend on the execution env.

thus, the match query in this test case may have multiple correct but different results.

consider changing the test case.

@czpmango
Copy link
Contributor

This is cypher's doc https://neo4j.com/docs/cypher-manual/current/execution-plans/shortestpath-planning/

Please test neo4j behavior and align with @nevermore3 the original intention of the implementation.

@czpmango
Copy link
Contributor

Fwiw, you can refer to this post to understand the difference between shortestpath pattern and allshortestpath pattern.
Difference between shortestpath and allshortestpaths in cypher

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

No branches or pull requests

4 participants