Skip to content

Conversation

@jeffreylovitz
Copy link
Contributor

Fix assertion error and invalid memory accesses described at RedisGraph/redisgraph-py#57

@swilly22
Copy link
Contributor

We need to add an AST validation

match (a) where a.v in 3 return a

Neo.ClientError.Statement.SyntaxError: Type mismatch: expected List<T> but was Integer (line 1, column 32 (offset: 31))
"explain match (a) where a.v in 3 return a"
GRAPH.EXPLAIN g "match (a) where a.v in 3 return a"
1) "Results"
2) "    Project"
3) "        Filter"
4) "            All Node Scan | (a)"

@swilly22 swilly22 merged commit 4089d95 into master Nov 12, 2019
@jeffreylovitz jeffreylovitz deleted the index-string-ownership branch February 6, 2020 22:52
pnxguide pushed a commit to CMU-SPEED/RedisGraph that referenced this pull request Mar 22, 2023
* Persist volatile strings when converting IN functions for index scans

* Remove dangerous assertion (captured by run-time error)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants