jamesfredley commented on code in PR #16583:
URL: https://github.com/apache/grails-core/pull/16583#discussion_r4411563339
##########
grails-data-neo4j/core/src/main/groovy/org/grails/datastore/gorm/neo4j/engine/Neo4jQuery.groovy:
##########
@@ -335,6 +335,11 @@ class Neo4jQuery extends Query {
@Override
@CompileStatic
CypherExpression handle(GraphPersistentEntity entity, Query.IsNull
criterion, CypherBuilder builder, String prefix) {
+ PersistentProperty prop =
entity.getPropertyByName(criterion.property)
Review Comment:
CastMember is a relationship. A null check on from or to enters this branch,
and the query already binds that relationship as r. The pattern then uses r as
a node, such as (r)-[]->(:Movie), and Neo4j rejects the query. Keep a null
check on an already-bound endpoint on that variable. A node association such as
owner stays NOT <relationship>.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
