[TINKERPOP-408] Neo4jHelpers.isDeleted(Node) should use getPropertyKeys() instead of getLabels().next()

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Done
• Fix Version/s: 3.0.0-incubating
• Component/s: neo4j
• Assignee: Unassigned
• Reporter: Florian Georg
• Created: 10/Dec/14 22:44
• Updated: 24/Mar/15 12:51
• Resolved: 24/Mar/15 12:51

Description

If a Node without any labels (in my case, from an upgraded Neo4j 1.x database) is passed to this method, the call to next() throws a NoSuchElementException. When it is called from a traversal (e.g. by g.V()) it is interpreted as terminating the traversal. Unfortunately, removing the call to next does not result in an IllegalStateException for a deleted node, but it appears that Node.getPropertyKeys() throws an IllegalStateException on a deleted node, even if the node had no properties before it was deleted. Also see issue TINKERPOP3-131.