You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2017/11/27 12:54:02 UTC
[1/4] tinkerpop git commit: TINKERPOP-1832 - Fix bug where
TraversalHelper.replaceStep sets the steps' previousStep incorrectly. It
needs to remove the replaced step before inserting the new step.
Repository: tinkerpop
Updated Branches:
refs/heads/master 7463b3ae3 -> bb67594cb
TINKERPOP-1832 - Fix bug where TraversalHelper.replaceStep sets the steps' previousStep incorrectly.
It needs to remove the replaced step before inserting the new step.
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/8bc4b36f
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/8bc4b36f
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/8bc4b36f
Branch: refs/heads/master
Commit: 8bc4b36f52d12e37b7d8f37f7d2240caa233106f
Parents: ec1cbda
Author: pieter <pi...@riseup.net>
Authored: Wed Nov 22 10:56:43 2017 +0200
Committer: pieter <pi...@riseup.net>
Committed: Wed Nov 22 10:56:43 2017 +0200
----------------------------------------------------------------------
.../process/traversal/util/TraversalHelper.java | 5 +++--
.../process/util/TraversalHelperTest.java | 19 +++++++++++++++++++
2 files changed, 22 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/8bc4b36f/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalHelper.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalHelper.java b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalHelper.java
index fd803e4..eda836a 100644
--- a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalHelper.java
+++ b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalHelper.java
@@ -173,8 +173,9 @@ public final class TraversalHelper {
* @param traversal the traversal on which the action will occur
*/
public static <S, E> void replaceStep(final Step<S, E> removeStep, final Step<S, E> insertStep, final Traversal.Admin<?, ?> traversal) {
- traversal.addStep(stepIndex(removeStep, traversal), insertStep);
- traversal.removeStep(removeStep);
+ final int i;
+ traversal.removeStep(i = stepIndex(removeStep, traversal));
+ traversal.addStep(i, insertStep);
}
public static <S, E> Step<?, E> insertTraversal(final Step<?, S> previousStep, final Traversal.Admin<S, E> insertTraversal, final Traversal.Admin<?, ?> traversal) {
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/8bc4b36f/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/util/TraversalHelperTest.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/util/TraversalHelperTest.java b/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/util/TraversalHelperTest.java
index 23eaa39..df22cd2 100644
--- a/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/util/TraversalHelperTest.java
+++ b/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/util/TraversalHelperTest.java
@@ -32,6 +32,7 @@ import org.apache.tinkerpop.gremlin.process.traversal.step.filter.LambdaFilterSt
import org.apache.tinkerpop.gremlin.process.traversal.step.filter.PathFilterStep;
import org.apache.tinkerpop.gremlin.process.traversal.step.filter.TraversalFilterStep;
import org.apache.tinkerpop.gremlin.process.traversal.step.filter.WhereTraversalStep;
+import org.apache.tinkerpop.gremlin.process.traversal.step.filter.NotStep;
import org.apache.tinkerpop.gremlin.process.traversal.step.map.FlatMapStep;
import org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesStep;
import org.apache.tinkerpop.gremlin.process.traversal.step.map.TraversalFlatMapStep;
@@ -47,6 +48,8 @@ import org.apache.tinkerpop.gremlin.structure.util.empty.EmptyGraph;
import org.junit.Test;
import org.mockito.Mockito;
+import java.util.List;
+import java.util.Optional;
import java.util.Set;
import java.util.stream.Collectors;
@@ -66,6 +69,22 @@ import static org.junit.Assert.assertTrue;
public class TraversalHelperTest {
@Test
+ public void shouldSetPreviousStepToEmptyStep() {
+ final Traversal.Admin<?, ?> traversal = __.V().out().asAdmin();
+ //transform the traversal to __.V().not(out())
+ //the VertexStep's previousStep should be the EmptyStep
+ Optional<VertexStep> vertexStepOpt = TraversalHelper.getFirstStepOfAssignableClass(VertexStep.class, traversal);
+ assertTrue(vertexStepOpt.isPresent());
+ Traversal.Admin<?,?> inner = __.start().asAdmin();
+ inner.addStep(0, vertexStepOpt.get());
+ TraversalHelper.replaceStep(vertexStepOpt.get(), new NotStep<>(__.identity().asAdmin(), inner), traversal);
+ List<VertexStep> vertexSteps = TraversalHelper.getStepsOfAssignableClassRecursively(VertexStep.class, traversal);
+ assertEquals(1, vertexSteps.size());
+ VertexStep vertexStep = vertexSteps.get(0);
+ assertTrue("Expected the previousStep to be an EmptyStep, found instead " + vertexStep.getPreviousStep().toString(),vertexStep.getPreviousStep() == EmptyStep.instance());
+ }
+
+ @Test
public void shouldIdentifyLocalChildren() {
final Traversal.Admin<?, ?> localChild = __.as("x").select("a", "b").by("name").asAdmin();
new LocalStep<>(new DefaultTraversal(), localChild);
[2/4] tinkerpop git commit: Merge branch 'pr-756' into tp32
Posted by sp...@apache.org.
Merge branch 'pr-756' into tp32
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/41806595
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/41806595
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/41806595
Branch: refs/heads/master
Commit: 41806595c6a2094a8422fab0e6154d557aa91b3a
Parents: 691ee9b 8bc4b36
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Mon Nov 27 07:34:28 2017 -0500
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Mon Nov 27 07:34:28 2017 -0500
----------------------------------------------------------------------
.../process/traversal/util/TraversalHelper.java | 5 +++--
.../process/util/TraversalHelperTest.java | 19 +++++++++++++++++++
2 files changed, 22 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
[3/4] tinkerpop git commit: Updated changelog CTR
Posted by sp...@apache.org.
Updated changelog CTR
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/072b0821
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/072b0821
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/072b0821
Branch: refs/heads/master
Commit: 072b08214d7e9e3d33d8ef40bfbbd1e6f746ea56
Parents: 4180659
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Mon Nov 27 07:36:05 2017 -0500
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Mon Nov 27 07:36:14 2017 -0500
----------------------------------------------------------------------
CHANGELOG.asciidoc | 1 +
1 file changed, 1 insertion(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/072b0821/CHANGELOG.asciidoc
----------------------------------------------------------------------
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
index 5ba7c70..02c87ca 100644
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@ -30,6 +30,7 @@ image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
* `TraversalVertexProgram` `profile()` now accounts for worker iteration in `GraphComputer` OLAP.
* Returned the `Builder` instance from the `DetachedEdge.Builder` methods of `setOutE` and `setOutV`.
* Added test framework for GLVs.
+* Fixed bug in `TraversalHelper.replaceStep()` where the step being replaced needed to be removed prior to the new one being added.
* Added alias support in the .NET `DriverRemoteConnection`.
* Added a test for self-edges and fixed `Neo4jVertex` to provided repeated self-edges on `BOTH`.
* Better respected permissions on the `plugins.txt` file and prevented writing if marked as read-only.
[4/4] tinkerpop git commit: Merge branch 'tp32'
Posted by sp...@apache.org.
Merge branch 'tp32'
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/bb67594c
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/bb67594c
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/bb67594c
Branch: refs/heads/master
Commit: bb67594cbfebbbe325862c0d9950537f5019b15d
Parents: 7463b3a 072b082
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Mon Nov 27 07:36:23 2017 -0500
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Mon Nov 27 07:36:23 2017 -0500
----------------------------------------------------------------------
CHANGELOG.asciidoc | 1 +
.../process/traversal/util/TraversalHelper.java | 5 +++--
.../process/util/TraversalHelperTest.java | 19 +++++++++++++++++++
3 files changed, 23 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/bb67594c/CHANGELOG.asciidoc
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/bb67594c/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalHelper.java
----------------------------------------------------------------------