GitHub user pietermartin opened a pull request:
https://github.com/apache/tinkerpop/pull/751
TINKERPOP-1832: TraversalHelper.replaceStep sets previousStep to the wrong
step
https://issues.apache.org/jira/browse/TINKERPOP-1832
`TraversalHelper.replaceStep` needs to first remove the step before adding
in the new step. Else it gets its `previousStep` pointer wrong.
Added `TraversalHelperTest.shouldSetPreviousStepToEmptyStep` which was the
scenario that originally made me aware of the issue.
Ran `mvn clean install` and tested Sqlg's tests with the fix.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pietermartin/tinkerpop TINKERPOP-1832
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tinkerpop/pull/751.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 #751
----
commit d40381c337ade06cf02388d014b7cc6d4ac8d7a1
Author: pieter <[email protected]>
Date: 2017-11-19T10:17:50Z
TINKERPOP-1832 - Fix bug where TraversalHelper.replaceStep sets the steps'
previousStep incorrectly.
It needs to remove the replaced step before inserting the new step.
----
---
