Skip to content
This repository has been archived by the owner on Oct 28, 2019. It is now read-only.

Greycat: GraphNodeWrapper equality should consider world+time+ID, not just ID #84

Open
agarciadom opened this issue May 1, 2019 · 0 comments
Assignees

Comments

@agarciadom
Copy link
Member

While writing a query, we noticed that if we compare a node in EOL with another version of the same node, it is reported to be equal. This should not be the case: GreycatNode explicitly compares world, time and ID in its equals method.

The GraphNodeWrapper is not reusing the equals method of the IGraphNode class. This should be revised, while making sure we preserve the same ID-based comparison that we had for Neo4j and OrientDB.

@agarciadom agarciadom self-assigned this May 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant