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

Crash encountered about same relationship variable in pattern #465

Open
gkorland opened this issue Oct 3, 2023 · 3 comments · May be fixed by #498
Open

Crash encountered about same relationship variable in pattern #465

gkorland opened this issue Oct 3, 2023 · 3 comments · May be fixed by #498
Assignees
Labels
bug Something isn't working

Comments

@gkorland
Copy link
Contributor

gkorland commented Oct 3, 2023

ref: RedisGraph/RedisGraph#3206

I execute a query on my empty RedisGraph and get a crash.

Redis version:7.2.0
RedisGraph version: v2.12.10
operating system: windows 11
API:Cypher version 9
query:WITH NULL AS a0 MATCH ()-[a0]-()-[a0]-() RETURN 1
error message:"Reached the max retries per request limit (which is 1). Refer to "maxRetriesPerRequest" option for details."

Actual behavior:
The query failed and the redisgraph container exited abnormally with the error message.

@gkorland gkorland added the bug Something isn't working label Oct 3, 2023
@gkorland
Copy link
Contributor Author

=== REDIS BUG REPORT START: Cut & paste starting from here ===
120212:M 28 Oct 2023 19:14:13.493 # === ASSERTION FAILED ===
120212:M 28 Oct 2023 19:14:13.493 # ==> /home/guy/workspace/FalkorDB/src/graph/query_graph.c:72 'QueryGraph_GetEdgeByAlias(qg, alias) == ((void *)0)' is not true

------ STACK TRACE ------

Backtrace:
./bin/linux-x64-debug/src/falkordb.so(+0x4fba21)[0x7ff5a3afba21]
./bin/linux-x64-debug/src/falkordb.so(QueryGraph_AddPath+0x12c)[0x7ff5a3afc7de]
./bin/linux-x64-debug/src/falkordb.so(BuildQueryGraph+0x3e5)[0x7ff5a3afcfec]
./bin/linux-x64-debug/src/falkordb.so(ExecutionPlan_PopulateExecutionPlan+0x55)[0x7ff5a39d3edb]
./bin/linux-x64-debug/src/falkordb.so(+0x3d4815)[0x7ff5a39d4815]
./bin/linux-x64-debug/src/falkordb.so(+0x3d4955)[0x7ff5a39d4955]
./bin/linux-x64-debug/src/falkordb.so(ExecutionPlan_FromTLS_AST+0x56)[0x7ff5a39d512b]
./bin/linux-x64-debug/src/falkordb.so(ExecutionCtx_FromQuery+0x238)[0x7ff5a39900c2]
./bin/linux-x64-debug/src/falkordb.so(_query+0x81)[0x7ff5a3987aed]
./bin/linux-x64-debug/src/falkordb.so(Graph_Query+0x21)[0x7ff5a3987d4a]
./bin/linux-x64-debug/src/falkordb.so(+0x648b03)[0x7ff5a3c48b03]
/lib/x86_64-linux-gnu/libc.so.6(+0x8f6ba)[0x7ff5a6c8f6ba]
/lib/x86_64-linux-gnu/libc.so.6(+0x11e0d0)[0x7ff5a6d1e0d0]

@swilly22
Copy link
Contributor

swilly22 commented Nov 4, 2023

@AviAvni
I'm not sure how should we treat bounded edges.

@AviAvni
Copy link
Contributor

AviAvni commented Nov 5, 2023

neo4j allow this and return empty result try this query in the neo4j browser and see the warning

@swilly22 swilly22 linked a pull request Nov 6, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants