GitHub user okram opened a pull request:
https://github.com/apache/incubator-tinkerpop/pull/199
TINKERPOP-1083: Traversal needs a hashCode() and equals() definition.
https://issues.apache.org/jira/browse/TINKERPOP-1083
We had a lot of inconsistencies around `hashCode()`/`equals()` for
`Traversal`. Moreover, I pretty much gutted a crap load of pointless internals
in `AbstractLambdaTraversal` (much simpler, safer, and more efficient).
`mvn clean install` as well as Spark integration tests (and the first 10
minutes of Giraph integration tests).
VOTE +1.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/incubator-tinkerpop TINKERPOP-1083
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-tinkerpop/pull/199.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #199
----
commit 9851241104eb738cd2646e50cdc9f5a424775ac5
Author: Marko A. Rodriguez <[email protected]>
Date: 2016-01-13T15:23:44Z
Greatly simplified/sped-up AbstractLambdaTraversal. It doesn't need graph,
strategy, parent, etc. references and all the complexities of cloning that that
stuff entails. Next, it now has a consistent equals()/hashCode() model.
Moreover, that hashCode()/equals() model that was only in DefaultGraphTraversal
has now been moved up to DefaultTraversal where it should have been to begin
with. Added proper TraverserRequirements to AbstractLambdaTraversal (hasnt bit
us yet, but it could).
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
