Github user robertdale commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/699#discussion_r136983503
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/traverser/B_O_Traverser.java
---
@@ -62,10 +62,12 @@ public void setStepId(final String stepId) {
this.future = stepId;
}
+ protected final boolean equals(final B_O_Traverser other) {
+ return super.equals(other) && other.future.equals(this.future);
+ }
+
@Override
--- End diff --
I agree with @okram. There should be an explicit `hashCode()` so it doesn't
look like it is mistakenly forgotten.
---
